Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Improving aria-label usage based on context #7492

Closed
wavvves opened this issue Oct 26, 2022 · 0 comments · Fixed by #8099
Closed

Improving aria-label usage based on context #7492

wavvves opened this issue Oct 26, 2022 · 0 comments · Fixed by #8099
Labels
focus: accessibility The issue/PR is related to accessibility. type: enhancement The issue is a request for an enhancement. type: good first issue The issue is a good candidate for the first community contribution/for a newcomer to the team.

Comments

@wavvves
Copy link
Contributor

wavvves commented Oct 26, 2022

While doing a pair-programming session on an unrelated task, I was left wondering how good our use of aria-label for accessibility purposes is and how it can be improved.

Best way to demonstrate this is with a quick example.

The “Remove item” action in the Cart block currently has no aria-label value; using Voice Over, it reads out only as the text it has. The shortcode version uses an “X” as the button text and has an aria-label value of “Remove this item.”. Both possess no labels. This is a bit generic without context. A user might not be sure what item is being removed from the cart.

I took a look at Amazon, for example, on the same functionality, and they provide context:

image

Apple.com does it via hidden text:

image

farfetch.com

image

(I also tried nike.com, but their cart was broken due to a CORS issue. Oopsie 😂)

Since we probably have multiple places where providing context would be an enhancement, a good suggestion about what could be done to improve on this was given on slack by @tarhi-saad:

Maybe a similar effort in Call to action to improve our public-facing text can be made here as well! (e.g., create a document with a list of all aria-labels text, propose a more clear alternative, etc.)

@wavvves wavvves added type: enhancement The issue is a request for an enhancement. focus: accessibility The issue/PR is related to accessibility. type: good first issue The issue is a good candidate for the first community contribution/for a newcomer to the team. labels Oct 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
focus: accessibility The issue/PR is related to accessibility. type: enhancement The issue is a request for an enhancement. type: good first issue The issue is a good candidate for the first community contribution/for a newcomer to the team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant