-
Notifications
You must be signed in to change notification settings - Fork 482
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
A11Y-7: Add alt text to careers page images #49513
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one typo :)
@@ -7255,6 +7255,8 @@ | |||
|
|||
language_label: "Language" | |||
|
|||
careers_page_code_pride_alt_text: "Two female-presenting employees smile while holding the ends of a Code.org-branded rainbow banner. Behind them, several people are wearing Code.org shirts and waving rainbow flags while walking alongside a multi-colored Volkswagon bug." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: should be "Volkswagen"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow TIL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
@@ -7255,6 +7255,8 @@ | |||
|
|||
language_label: "Language" | |||
|
|||
careers_page_code_pride_alt_text: "Two female-presenting employees smile while holding the ends of a Code.org-branded rainbow banner. Behind them, several people are wearing Code.org shirts and waving rainbow flags while walking alongside a multi-colored Volkswagon bug." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like that you added this description!
Nit: I think female
has a biological connotation that I don't think you're intending. Maybe feminine-presenting
or woman-presenting
? Language for gender is evolving which makes this hard!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heck yes! Thank you for incorporating feedback and writing a great description!
This PR adds alt text for these two images. I decided to give the all-employee photo empty alt text because it doesn't feel like it conveys anything in particular, but the Code.org pride photo feels worth explaining.
Links
Jira ticket: https://codedotorg.atlassian.net/browse/A11Y-7
Testing story
Tested locally with VoiceOver (see output below).