-
Notifications
You must be signed in to change notification settings - Fork 625
Aria issues #941
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
Comments
Forgive me. I have replicated the errors using Axe scanner on the demo. If you scan the demo you'll see the WCAG 2.0 AA aria violations. https://choices-js.github.io/Choices/ https://chrome.google.com/webstore/detail/axe-devtools-web-accessib/lhdoppojpmngadmnindnejefpokejbdd |
Hey all! Any news here? |
The v11.0.x updates did see some aria related changes but it isn't completely fixed. Should be better than it used to be. |
Additionally, if we're entertaining minor a11y changes, a typical Choices dropdown will add the attribute According to a11y best practice, it's recommended to not add the role="textbox" to an text or search element . Rather, screen readers and other assistive technologies already infer that it's a textbox from the element itself. P.S. Thanks to the collaborators. Choices.js is a great plugin. |
That should be a simple PR (looks to be just one spot adding the html attribute, and then updating a unit-test). Failing that can you link to the relevant a11y best practice so the change could be documented? |
I believe this is the precise link on the w3C aria documentation:
Additionally, I find the Mozilla MDN documentation to be helpful: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/textbox_role
In other words, both sources imply this should only be used on non-input elements. We'd run an accessibility scan against our site, and it did in fact trigger a "attribute is not allowed" error. I just submitted a PR, please review, much appreciated. |
Remove `role="textbox"` from search input, per a11y practices. #941
@mlinnetz that has been merged in. I don't have an ETA for the next release but the minified js files have also been updated if you require those, otherwise just building against |
I am having to issues validating the code and I don't know how to do it with choices.
ARIA items do not have accessible names
Elements with an ARIA [role] that require children to contain a specific [role] are missing some or all of those required children
Anybody knows how I can implement this on choices?
The text was updated successfully, but these errors were encountered: