-
Notifications
You must be signed in to change notification settings - Fork 220
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
📚 Documentation: multiple <nav> elements should be labelled [a11y][TheA11y100] #294
Comments
Would love to work on this! |
@divyanshkul Assigned it to you. We can add the |
Hi @TGlide. Hacktoberfest is over, and it seems that the assignee has not worked on this issue. If you're open, I'd love to work on this. |
Go ahead! |
@CBID2 are you working on this issue? |
Yes |
Issue
There are multiple
<nav>
elements on each page. When you have more than one<nav>
you should add a label to each.Why it matters
When you have more than one
<nav>
element you should add a label to each to aid navigation with assistive tech and to differentiate them, otherwise when navigate via landmarks it can be confusing which navigation a user is targetting.Suggested fix
Add
aria-label
s to each of the<nav>
elements to identify their purposes.For example:
Nav at the top could be
<nav aria-label="main">
and the nav at the side could be<nav aria-label="docs">
(these could be better, if someone can think of more descriptive names for each navigation element. You do not need to say "navigation" as part of the text.)Relevant Guidance
multiple navigation landmarks
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: