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

Fix missing "switch" role on OnyxSwitch #754

Closed
JoCa96 opened this issue Mar 25, 2024 · 0 comments
Closed

Fix missing "switch" role on OnyxSwitch #754

JoCa96 opened this issue Mar 25, 2024 · 0 comments
Assignees
Labels
bug Something isn't working dev Requires technical expertise

Comments

@JoCa96
Copy link
Contributor

JoCa96 commented Mar 25, 2024

We are missing the switch role for out OnyxSwitch.

Implementation details

Eslint rule is incorrect, see vue-a11y/eslint-plugin-vuejs-accessibility#932

@JoCa96 JoCa96 self-assigned this Mar 25, 2024
@JoCa96 JoCa96 added bug Something isn't working dev Requires technical expertise labels Mar 25, 2024
JoCa96 added a commit that referenced this issue Mar 25, 2024
Relates to #754 

Add optional, but semantically helpful `role="switch"` to the
OnyxSwitch.
The “switch” role can be used directly on a native `<input
type="checkbox" />` as shown in the example on
https://www.w3.org/WAI/ARIA/apg/patterns/switch/examples/switch-checkbox/.
@JoCa96 JoCa96 closed this as completed Mar 25, 2024
@mj-hof mj-hof added this to the Selection controls milestone Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dev Requires technical expertise
Projects
Status: Done
Development

No branches or pull requests

2 participants