Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Focused links can be covered by the table of contents button on mobile #356

Closed
tombye opened this issue Sep 6, 2024 · 1 comment
Closed

Comments

@tombye
Copy link
Contributor

tombye commented Sep 6, 2024

What should change

The table of contents button should be changed for smaller screens so it can't overlap links when they are focused.

User need

If you rely on the keyboard, you need to see what's in focus at all times. If a link gets hidden, it also breaks success criterion 2.4.11, Focus not obscured (minimum) and success criterion 2.4.12, Focus not obscured (enhanced).

For example, see the link to the 'GOV.UK kubernetes manual' on https://docs.publishing.service.gov.uk/ receiving focus while hidden:

mobile_tech_docs_template_focus_obscured_fail

@tombye
Copy link
Contributor Author

tombye commented Sep 19, 2024

We added a fix for this in #360 so I'm marking this as closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant