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

Drop alt attribute of source favicons #978

Merged
merged 1 commit into from
Oct 3, 2017
Merged

Drop alt attribute of source favicons #978

merged 1 commit into from
Oct 3, 2017

Conversation

jtojnar
Copy link
Member

@jtojnar jtojnar commented Aug 17, 2017

The favicons in the source list contained the title of the feed in their alt attribute. This was redundant since the title is already shown next to the item, resulting in the title being read multiple times by accessibility technologies. It was also displayed over the title when the favicon file was missing.

Closes: #975

@jtojnar jtojnar added the ux label Aug 17, 2017
The favicons in the source list contained the title of the feed in
their alt attribute. This was redundant since the title is already
shown next to the item, resulting in the title being read multiple
times by accessibility technologies. It was also displayed over
the title when the favicon file was missing.

Closes: #975
@jtojnar jtojnar merged commit 697697c into master Oct 3, 2017
@jtojnar jtojnar deleted the source-alt branch October 3, 2017 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant