-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
fix(components): [checkbox] indeterminate checkbox a11y fix #14322
fix(components): [checkbox] indeterminate checkbox a11y fix #14322
Conversation
👋 @Karolis-Stoncius, thank you for contributing element-plus. |
Hello @Karolis-Stoncius, thank you for contributing to element-plus, please see our guideline to see how to make contribution |
This is the same PR as this, I had to delete my fork, so I created a new PR. |
🧪 Playground Preview: https://element-plus.run/?pr=14322 |
@btea when will this be merged and released? |
Don't worry, it still needs to be reviewed and confirmed by others. |
Fixes a bunch of a11y issues when a checkbox is indeterminate. Here's a playground , select one of the items and scan for a11y issues (I'm using axe DevTools). This PR fixes these a11y issues caused by an indeterminate checkbox:
https://dequeuniversity.com/rules/axe/4.7/aria-hidden-focus?application=AxeChrome
https://dequeuniversity.com/rules/axe/4.7/aria-toggle-field-name?application=AxeChrome
https://dequeuniversity.com/rules/axe/4.7/nested-interactive?application=AxeChrome
Please make sure these boxes are checked before submitting your PR, thank you!
dev
branch.Description
🤖 Generated by Copilot at d7683ca
Improve checkbox component accessibility and functionality. Use native
indeterminate
attribute and remove unnecessary attributes fromspan
andinput
elements incheckbox.vue
.Related Issue
Fixes #___.
Explanation of Changes
🤖 Generated by Copilot at d7683ca
indeterminate
attribute to input element for partial selection state (link, link)aria-hidden
attribute from input element to avoid accessibility issues (link, link)