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

Checked checkboxes should have checked="true" and not aria-checked="true" #2514

Open
2 tasks done
coliff opened this issue May 10, 2024 · 0 comments
Open
2 tasks done
Assignees
Labels
accessibility bug Something is wrong and needs to be addressed

Comments

@coliff
Copy link
Member

coliff commented May 10, 2024

Prerequisites

Describe the issue

The current behaviour is problematic.

  • Checked checkboxes should have checked="true" (and not aria-checked="true")
  • Unchecked checkboxes should have checked="false" (and not aria-checked="false")

Deque university has more details on this:
https://dequeuniversity.com/rules/axe/4.9/aria-conditional-attr?application=AxeEdge

Reduced test cases

Run Axe Core scan on: https://coliff.github.io/modus-web-components-example/

image

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Microsoft Edge

What is the issue regarding ?

@trimble-oss/modus-web-components

What version of npm package are you using ?

latest

Priority

Medium

What product/project are you using Modus Components for ?

Modus

What is your team/division name ?

Modus

Are you willing to contribute ?

Yes

Are you using Modus Web Components in production ?

No response

@coliff coliff added bug Something is wrong and needs to be addressed accessibility labels May 10, 2024
@coliff coliff self-assigned this May 10, 2024
@coliff coliff changed the title Checked checkboxes should have aria-checked="false" and not aria-checked="false" Checked checkboxes should have checked="true" and not aria-checked="true" May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug Something is wrong and needs to be addressed
Projects
Status: In Review
Development

No branches or pull requests

1 participant