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-current="page" to the active link #3144

Closed
phiter opened this issue Mar 13, 2020 · 1 comment
Closed

Add aria-current="page" to the active link #3144

phiter opened this issue Mar 13, 2020 · 1 comment

Comments

@phiter
Copy link

@phiter phiter commented Mar 13, 2020

What problem does this feature solve?

Users that can see the page can have a nice experience of knowing which link is currently active just by looking at a styled active link with the activeClass.

But visually impaired users, those who rely on screen readers, currently have no way of knowing that the currently selected link is the active link.

To fix this, aria-current="page" adds an extra information to the screen reader that the currently focused link is the same as the current page, so if we add that together with activeClass we'll be aiding both sighted and blind users.

What does the proposed API look like?

Basically, when we add activeClass also pass aria-current="page" to the generated anchor.

@posva

This comment has been minimized.

Copy link
Member

@posva posva commented Mar 13, 2020

Duplicate of #2116

@posva posva marked this as a duplicate of #2116 Mar 13, 2020
@posva posva closed this Mar 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.