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.

Sign up for GitHub

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

Jump to bottom

Redundant use of word "Navigation" in ARIA label #253

Closed
krishnabodawala opened this issue Sep 24, 2023 · 1 comment · Fixed by #256
Closed

Redundant use of word "Navigation" in ARIA label #253

krishnabodawala opened this issue Sep 24, 2023 · 1 comment · Fixed by #256
Labels
bug Something isn't working

Comments

@krishnabodawala
Copy link

krishnabodawala commented Sep 24, 2023 •
edited
Loading

Description

Using the word "Navigation" to the ARIA label for the element marked up as HTML navigation, makes the screen reader announce it twice , for a screen reader user, it can get tiresome.

Note

The relevant WCAG success criteria is1.3.1
Reference Accessibility Audit #245

Background

Test performed by Krishna Bodawala Accessibility Consultant Iota
@krishnabodawala krishnabodawala added the bug Something isn't working label Sep 24, 2023
@smythp smythp mentioned this issue Sep 27, 2023
Merged
@smythp
Copy link
Collaborator

smythp commented Sep 27, 2023

@krishnabodawala Pretty sure this is resolved, but can you double check this preview to see if it fixes the main navigation redundency?

https://deploy-preview-256--docssigstore.netlify.app/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed aria label in main nav to resolve redundancy smythp/sigstore-docs
2 participants
@smythp @krishnabodawala