Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
Hide flair from screen readers
Browse files Browse the repository at this point in the history
To have less noise when they run over the sender profile.

See element-hq/element-web#9747
  • Loading branch information
turt2live committed May 22, 2019
1 parent 8b90178 commit 5ae23fd
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion src/components/views/elements/Flair.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,18 @@ class FlairAvatar extends React.Component {
const tooltip = this.props.groupProfile.name ?
`${this.props.groupProfile.name} (${this.props.groupProfile.groupId})`:
this.props.groupProfile.groupId;

// Note: we hide flair from screen readers but ideally we'd support
// reading something out on hover. There's no easy way to do this though,
// so instead we just hide it completely.
return <img
src={httpUrl}
width="16"
height="16"
onClick={this.onClick}
title={tooltip} />;
title={tooltip}
aria-hidden={true}
/>;
}
}

Expand Down

0 comments on commit 5ae23fd

Please sign in to comment.