-
-
Notifications
You must be signed in to change notification settings - Fork 819
Conversation
…-org/matrix-react-sdk into t3chguy/room-list/14466
Overall, looks good. Some minor design comments: Explore icon and text seem to be different link elements.Really small thing but it makes them a bit harder to use. If difficult, don't think it's super important at all. Font-weightsTrying to minimise the amount of font-weights used arbitrarily, unfortunately Figma is behind in this regard. Sorry about that Margins.mx_LeftPanel .mx_LeftPanel_roomListContainer .mx_LeftPanel_roomListFilterCount margin-top 14px works better than 16px in practice. I'll make a note to update figma later this week. Not necessarily relevant to this issue but noting anyway:
|
Agreed, that was done during rebranding. |
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.
lgtm
Based on #5118Requires element-hq/element-web#14983
Fixes element-hq/element-web#14466
Build at http://riots.im/adhoc/room-list-filtering1/