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
Change alt
attributes to be empty
#943
base: main
Are you sure you want to change the base?
Conversation
Agree with the complete redundancy here; but I'm curious if we should add an alt of "flag" to the image? This would result with an AT reading of "Image ... Flag ... USA, etc". I think I prefer your approach but would like @josepharhar to weigh in since he's primarily making the |
These shouldnt have alt values. Flag isn’t useful here unless the intent is to choose a flag |
Not a blocker here but we should add label elements to each of the selectlists |
Hey, please can this PR be merged now? Since it has been months |
@mfreed7 or @gregwhitworth would be good if one of you could approve so we can merge :) |
Based on this reply, this LGTM. |
For these images, the
alt
attribute should be empty because these images are decorational. Since all the info is already conveyed through the option's text.Currently, a screen reader would announce "flag of USA USA" as the accessible name, which is verbose.