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

Add shields to member list, move power label to text #4006

Merged
merged 8 commits into from
Jan 31, 2020

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Jan 30, 2020

This adds verification shields to the member list (when cross-signing is on) and also moves the power labels (admin / mod) to text on the tile.

member-list-shields

Fixes element-hq/element-web#11940

This displays verification status on each room member tile using the same shield
logic as elsewhere.

Part of element-hq/element-web#11940
Unclear why this is here, but removing as it feels quite glitchy, esp on
matrix.org where there's no second line to print.
The power label is shown to the right of the name, but hidden on hover.

Part of element-hq/element-web#11940
@jryans jryans requested a review from a team January 30, 2020 22:14
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this interact when someone inevitably translates the string to "Administrator of the room" or something similarly long?

src/components/views/rooms/EntityTile.js Show resolved Hide resolved
src/components/views/rooms/MemberTile.js Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User shields on avatars in the room member list
2 participants