-
-
Notifications
You must be signed in to change notification settings - Fork 52
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
A11Y error due to tabindex > 0 #118
Comments
Could probably just remove the tabindex all together and users can apply those as needed. |
I'm not sure, as a general A11Y rule, it should be |
The user can already set it, e.g. |
I'll change it to '0' |
Thanks for the fix @knownasilya |
Released in 5.3.1 |
Thanks for this awesome addon.
Below you will see an
A11Y
error related, it was catched byember-A11Y-testing
it seems an easy one to fix on 馃憠 x-toggle/component.jsI'm happy to create a PR if you agree this should be
tabindex: '0'
.Thanks
The text was updated successfully, but these errors were encountered: