Conversation
|
/backport to stable25 |
jotoeri
left a comment
There was a problem hiding this comment.
Hmm, isn't that worse, as the screenreader reads the name on the avatar and then once again on the actual full-name text?
Do you think we should add a separate label for the avatar @michaelnissenbaum? |
|
@JuliaKirschenheuter From my perspective, Avatar should not be implemented as a link because we already have a 'View Profile' link that provides the same functionality. Avatar should simply be implemented as an img element with an "alt" attribute set to an empty string, so that screen readers will ignore this element. However, if you really want the Avatar element to remain interactive and be displayed, then you can use the aria-label attribute with an "a"-element or the "alt" attribute of the image. |
There was a problem hiding this comment.
would be better to set Avatar of __name__? Or should we through away a link from avatar #36740 (comment)?
e37426d to
1748296
Compare
Makes sense UX-wise to have it clickable so updated the description! |
Signed-off-by: Christopher Ng <chrng8@gmail.com>
1748296 to
9d904b1
Compare

Summary
Add descriptive text to the contacts menu avatar
imgelementsChecklist