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

Add aria-hidden to non-active header and tabindex="-1" to its focusable elements #80

Open
feministclickback opened this issue Jul 25, 2024 · 0 comments

Comments

@feministclickback
Copy link

This is a suggestion for an A11Y improvement to comply with WCAG 2.2 AA.

The issue: the header is duplicate for non-sighted users because it is visually hidden, but not taken out of the accessibility tree. If it contains focusable elements such as a link, you can tab to them, but you can't see them, which is a problem for keyboard users.

Proposed fix:

  • Add aria-hidden="true" to non-active header
  • Add tabindex="-1" to any focusable elements in the non-active header

More information here: https://dequeuniversity.com/rules/axe/4.1/aria-hidden-focus

If you point me to where in the code this would be added, I can try and see if I can write a fix myself, but I'm a beginner so bear with me!

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

No branches or pull requests

1 participant