-
Notifications
You must be signed in to change notification settings - Fork 147
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
Aria-selected not being set (with fix) #566
Comments
In the default onConfirm for accessible autocomplete we try to set the 'selected' property. This is wrong. As per docs above selected is used to indicate an option selected by default on page load not the currently selected option. aria-selected on the other hand: Should be set. This can be set by changing the line highlighted: I can't do this right now so if anyone fancvies a low hanging fruit contribution and putting together a PR then there's a freebie. |
accessible-autocomplete/src/wrapper.js
Line 29 in a2e926f
Had a problem with this not being set properly, will come back to flesh out issue later
The text was updated successfully, but these errors were encountered: