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 to SelectItem #2828

Open
csmatt opened this issue Apr 10, 2024 · 0 comments
Open

Add aria-current to SelectItem #2828

csmatt opened this issue Apr 10, 2024 · 0 comments

Comments

@csmatt
Copy link

csmatt commented Apr 10, 2024

Feature request

Overview

Currently, the selected SelectItem does not have the aria-current attribute set to true. This attribute lets screen readers know which item is selected so it can relay that information to the user. NVDA is a screen reader that is free and will help a tester hear the difference. I understand that the spec should not require this, but the screen readers like JAWS and NVDA seem to. Another option, if possible, would be to allow access to the context form outside the component so that I can wrap SelectItem and pass it the attribute myself.

Examples in other libraries

Who does this impact? Who is this for?

Users who are navigating with a screen reader like JAWS or NVDA.

Additional context

It would just need the addition of aria-current={isSelected && isFocused} to SelectItem

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