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

Accessible button using ARIA #2024

Closed
Ryuno-Ki opened this issue Feb 2, 2021 · 2 comments · Fixed by #2238
Closed

Accessible button using ARIA #2024

Ryuno-Ki opened this issue Feb 2, 2021 · 2 comments · Fixed by #2238
Assignees

Comments

@Ryuno-Ki
Copy link
Collaborator

Ryuno-Ki commented Feb 2, 2021

The other day, @weboverhauls tweeted

Pro tip: for selected state of a button, use aria-pressed. (Don't use aria-checked nor aria-selected; I've seen this mistake 2-3 times in last few days...) https://w3.org/TR/wai-aria-1.1/#aria-pressed #webdev #a11y #aria #tips

I consider this important information and would like to see it on MDN web docs somewhere.

My first attempt was to look at https://developer.mozilla.org/en-us/docs/Web/Accessibility/ARIA/Roles/button_role but that already documents it (and does not mention aria-checked or aria-selected). We don't list ARIA attributes in isolation (there's an issue about the mixins in #1940, which is the reason behind, I guess).

So where would that information go to?

@rachelandrew
Copy link
Collaborator

@Ryuno-Ki the mixins issue isn't related, that is the Interface and not the HTML attributes. We don't, as you say, have individual attributes listed but that is unrelated to the work on the API side of things.

@ericwbailey do you have any thoughts on the issue raised?

@ericwbailey
Copy link
Member

This feels like it could be a subsection for Accessibility concerns for the button element to me.

@rachelandrew rachelandrew self-assigned this Feb 10, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants