Update RoomSummaryCard navigation links #11812
Conversation
germain-gg
added
the
T-Enhancement
There was a problem hiding this comment.
The new sidebar looks really good 🎨
Should the widgets item not also be a new menu item? It does now look a bit out of place.
Also without using the new room header you can now no longer reach the member list. Designs use tabs here. Maybe as an intermediate solution we could add a „People“ item?
| checked={isFavorite} | ||
| onClick={() => tagRoom(room, DefaultTagID.Favourite)} | ||
| /> | ||
| <MenuItem |
There was a problem hiding this comment.
The disabled state is not reflected in the UI. It still looks like, that you can click the item:
Tested in Matrix-HQ
There was a problem hiding this comment.
Fixed in compound 0.8.1
|
I created a new disabled menu item state. |
|
hey @weeman1337, this is ready for another pass. thanks! |
|
@weeman1337 the widgets change is captured here, under the 'next iteration' epic https://github.com/vector-im/wat-internal/issues/77 |
For element-hq/element-web#25883
This PR partially implements the new room details designs.
Not included in this PR:
Checklist
Here's what your changelog entry will look like:
✨ Features