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.

Sign up for GitHub

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

Jump to bottom

Consider removing aria-expanded from conditional radios #4974

Closed
joelanman opened this issue May 9, 2024 · 1 comment
Closed

Consider removing aria-expanded from conditional radios #4974

joelanman opened this issue May 9, 2024 · 1 comment
Labels
accessibility-concern Any bug, feature request or question about the accessibility of a portion of a product. 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) radios

Comments

@joelanman
Copy link
Contributor

joelanman commented May 9, 2024 •
edited

Description of the issue

aria-expanded on radios is not valid in the spec. According to this thread, it's potentially superfluous and only adds noise

So this is an issue to consider removing it from radios

Came up recently here:
@joelanman joelanman added awaiting triage Needs triaging by team 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) labels May 9, 2024
@romaricpascal romaricpascal added radios accessibility-concern Any bug, feature request or question about the accessibility of a portion of a product. and removed awaiting triage Needs triaging by team labels May 10, 2024
@romaricpascal
Copy link
Member

It's a duplicate of #979, so I'll close it to track things in a single place.

@romaricpascal romaricpascal closed this as not planned Won't fix, can't repro, duplicate, stale May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility-concern Any bug, feature request or question about the accessibility of a portion of a product. 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) radios
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@romaricpascal @joelanman