Skip to content

Provide alternative text for images where appropriate#9853

Merged
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:image-updates
Nov 13, 2020
Merged

Provide alternative text for images where appropriate#9853
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:image-updates

Conversation

@darkwing
Copy link
Copy Markdown
Contributor

Explanation:

At present, we aren't effectively using our alt attributes; when an alt attribute is missing, screen readers read the image URL, which isn't good. we should use empty alts to avoid that. I've also added text where I felt appropriate.

@darkwing darkwing requested a review from a team as a code owner November 11, 2020 15:47
@darkwing darkwing requested a review from brad-decker November 11, 2020 15:47
@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Copy link
Copy Markdown
Contributor

@brad-decker brad-decker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phenomenal work once again!

@darkwing darkwing merged commit ed19ce4 into MetaMask:develop Nov 13, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Nov 13, 2020
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 this pull request may close these issues.

2 participants