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

Add a context menu to rooms in the new room list #4743

Merged
merged 3 commits into from
Jun 10, 2020

Conversation

turt2live
Copy link
Member

For element-hq/element-web#13635

The tag options are not implemented out of concern for diff size.

This splits the context menu classes out to a new "iconized" style which is common across a number of context menus, including the UserMenu.

Some of the badge/sublist styles had to change to better accommodate the menu icon lining up.

This also contains the framework required for element-hq/element-web#13961

image

turt2live added 2 commits June 9, 2020 23:09
The tag options are not implemented out of concern for diff size. 

This splits the context menu classes out to a new "iconized" style which is common across a number of context menus, including the UserMenu.

Some of the badge/sublist styles had to change to better accommodate the menu icon lining up.

This also contains the framework required for element-hq/element-web#13961
@turt2live turt2live requested a review from a team June 10, 2020 05:14
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

lgtm otherwise

@turt2live turt2live merged commit 72bb89d into develop Jun 10, 2020
@turt2live turt2live deleted the travis/room-list/tile-menu branch June 10, 2020 13:46
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.

2 participants