POC for the new status indicator everywhere#34127
Draft
langleyd wants to merge 34 commits into
Draft
Conversation
1. Make avatar BIGGER! 2. Move status button out to its own component so we can reuse it
but using context menu rather than dropdown menu which almost looks right but not quite.
and styling tweaks
for customisable dropdown
as combobox role things don't just inherit from their inner text
because it doesn't need to be. Also rename accordingly.
including room settings one which wasn't really supposed to change although the avatar now takes up the space it's in - leaving it for design to check.
…profile Extends MSC4426 user status display (already in the timeline) to: - DM room list items: status emoji after the room name (RoomListItemViewSnapshot + RoomListItemViewModel fetch/UserProfileUpdate subscription) - DM room header: emoji + status text after the room name - Right panel user profile: emoji + status text between name and MXID - Member list: status emoji after the member name via DisambiguatedProfile (member-tile layout switched to grid so name truncates and emoji stays) Adds a shared UserStatusMessageView (emoji + text) and getUserStatusEmoji helper, extracts a reusable fetchUserStatus util from useUserStatus, and bumps the status emoji to body-lg per design. Also syncs the jest transformIgnorePatterns matrix-js-sdk entry from develop so unit tests run. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…us-everywhere # Conflicts: # apps/web/src/viewmodels/room-list/RoomListItemViewModel.ts
5 tasks
Member
|
Squash merging has confused git and caused the mass conflicts above so I've just re-cherry-picked the commit into a new branch: #34161 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
POC Show user status in DM room list, room header, member list & user profile
UserStatusMessageView+getUserStatusEmojihelper; extracts a reusablefetchUserStatusutilUserProfileUpdateupdatesbody-lgper design (also affects the timeline, which shares the component)feature_user_statuslabs flag; unit tests/snapshots updated and verified live in-appClaude assisted