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

Action buttons do not describe element being controlled #980

Open
kimisgold opened this issue Jul 7, 2022 · 4 comments
Open

Action buttons do not describe element being controlled #980

kimisgold opened this issue Jul 7, 2022 · 4 comments
Labels

Comments

@kimisgold
Copy link
Member

Our accessibility audit would like to see our action buttons to have more descriptive labels that include what is being affected by the action. For example, "Remove" should read "Remove 'Title' element". This would mean a lot of strings would need translation updates. I tried looking into aria roles that would fit the bill, but the closest I could find was aria-controls, which seems to have really poor support across screen readers. How do we want to address this, if at all?

@kimisgold kimisgold added the a11y label Jul 7, 2022
@zerocrates
Copy link
Member

zerocrates commented Jul 7, 2022

Where specifically is this being flagged?

We recently did work along these lines for the main edit forms... my recollection is that it used kind of a chain (not actually a chain but just multiple values) of aria-labelledby to build up more complex accessible labels from the existing labels. Or I should say we accepted a PR that did this, that is.

We also sometimes do things like using fieldsets or aria-group to provide context to repeated controls like that.

@kimisgold
Copy link
Member Author

Ah, maybe this is a pattern I forgot about and it's not been implemented across the plugins. I'm looking at issue 72 for Contribution.

@kimisgold
Copy link
Member Author

Ah right, I think it's this one? #931 I'll look to it for reference.

@zerocrates
Copy link
Member

Specific to contribution, you might not need lots of strings for this, so explicit aria-labels that use a "remove element %s" type pattern might just be the best move there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants