-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Wmwragg/mention state menu #1900
Conversation
…e and below the name
…e room name if too long
…ow. OnClick closes menu
…ention-state-menu # resolved Conflicts: # src/skins/vector/css/vector-web/views/rooms/RoomTooltip.css
…as they were causeing untraceable react errros for some reason
…s faded and disabled options
… it pasues breifly before closing, so that the user can see their state change has taken place.
…vatar, but don't show the mute icon
Can one of the admins verify this patch? |
…ch design in both normal and collapsed states
…d. Also search box gutters fixed
@@ -94,7 +94,7 @@ limitations under the License. | |||
background-color: #eaf5f0; | |||
|
|||
-webkit-flex: 0 0 210px; | |||
flex: 0 0 210px; | |||
flex: 0 0 235px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm, the -webkit-flex
is now out of sync with the flex
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that the -webkit-flex
is used anymore, but I've corrected anyway
LGTM modulo comment - thanks :) |
This closes #1841 and closes #1746. A companion branch matrix-org/matrix-react-sdk:wmwragg/mention-state-menu also needs to be merged.
Signed-off-by: William Wragg wm.wragg@gmail.com