GlAvatar should allow empty alt text
Problem
- Avatar
alt="avatar"
which is vague and unnecessary since the user name is included in the proceeding text.
CleanShot_2022-06-01_at_10.29.34 - Identicon avatars (the ones with a single text character) shouldn't be announced at all.
- The current prop defaults to
avatar
so you can't set an empty alt attribute
Solution
- Use a null value for image avatars, for example
alt=""
which will indicate the avatar is "decorational" and not meaningful content. - Prevent identicons from being announced by using
aria-hidden="true"
.
Edited by Sam Beckham