imterah/element-portable
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Hide user avatars from screen readers in group and room user lists.

Browse source 
Like in pill components, these only duplicate information for screen readers, so hide them.

Signed-off-by: Marco Zehe <marcozehe@mailbox.org>
This commit is contained in:
Marco Zehe 2020-02-28 12:40:43 +01:00
parent 5325b5677f
commit 41e3e266e4
3 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/components/views/groups/GroupMemberTile.js
View file

@ -59,7 +59,7 @@ export default createReactClass({
); );
const av = ( const av = (
<BaseAvatar <BaseAvatar aria-hidden="true"
name={this.props.member.displayname || this.props.member.userId} name={this.props.member.displayname || this.props.member.userId}
idName={this.props.member.userId} idName={this.props.member.userId}
width={36} height={36} width={36} height={36}

2
src/components/views/rooms/EntityTile.js
View file

@ -175,7 +175,7 @@ const EntityTile = createReactClass({
const BaseAvatar = sdk.getComponent('avatars.BaseAvatar'); const BaseAvatar = sdk.getComponent('avatars.BaseAvatar');
const av = this.props.avatarJsx || <BaseAvatar name={this.props.name} width={36} height={36} />; const av = this.props.avatarJsx || <BaseAvatar name={this.props.name} width={36} height={36} aria-hidden="true" />;
// The wrapping div is required to make the magic mouse listener work, for some reason. // The wrapping div is required to make the magic mouse listener work, for some reason.
return ( return (

2
src/components/views/rooms/MemberTile.js
View file

@ -208,7 +208,7 @@ export default createReactClass({
} }
const av = ( const av = (
<MemberAvatar member={member} width={36} height={36} /> <MemberAvatar member={member} width={36} height={36} aria-hidden="true" />
); );
if (member.user) { if (member.user) {