Skip to content
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

Implement changes to memberlist from feedback #29029

Merged
merged 8 commits into from
Jan 21, 2025

Conversation

MidhunSureshR
Copy link
Member

@MidhunSureshR MidhunSureshR commented Jan 20, 2025

Closes https://github.com/element-hq/wat-internal/issues/246

Following changes are made:

  1. Member tiles for three-pid invites never showed the (Invite) label. This is now fixed.
  2. Tiles for both three-pid invites and room member invites now includes an icon.
  3. There is now a separator between joined members and invited members.
  4. All invited members are shown at the bottom of the list.

- ThreePidInviteTile now contains an user label showing "(Invited)" and
  an email icon.
- RoomMemberTile now includes an icon similar to above.
- Refactors a bunch of code to make this change sensible.
Copy link
Member

Choose a reason for hiding this comment

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

What is the rule for the MXID being shown? Its very confusing there are two Bob's but only one is disambiguated.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I would also expect them both to be disambiguated. But this is a long standing bug: #468

@MidhunSureshR MidhunSureshR added this pull request to the merge queue Jan 21, 2025
Merged via the queue into develop with commit 1644169 Jan 21, 2025
33 checks passed
@MidhunSureshR MidhunSureshR deleted the midhun/new-memberlist-feedback-changes branch January 21, 2025 11:13
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Feb 6, 2025
https://build.opensuse.org/request/show/1242244
by user dheidler + anag+factory
- Fix build by breakig the bad esbuild library for good
  * Add break-esbuild-for-good.patch

- Version 1.11.91
  ## ✨ Features
  * Implement changes to memberlist from feedback (element-hq/element-web#29029). Contributed by @MidhunSureshR.
  * Add toast for recovery keys being out of sync (element-hq/element-web#28946). Contributed by @dbkr.
  * Refactor LegacyCallHandler event emitter to use TypedEventEmitter (element-hq/element-web#29008). Contributed by @t3chguy.
  * Add `Recovery` section in the new user settings `Encryption` tab (element-hq/element-web#28673). Contributed by @florianduros.
  * Retry loading chunks to make the app more resilient (https://github.com/element-hq/element-web/pul
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Feb 6, 2025
https://build.opensuse.org/request/show/1242243
by user dheidler + anag+factory
- Version 1.11.91
  ## ✨ Features
  * Implement changes to memberlist from feedback (element-hq/element-web#29029). Contributed by @MidhunSureshR.
  * Add toast for recovery keys being out of sync (element-hq/element-web#28946). Contributed by @dbkr.
  * Refactor LegacyCallHandler event emitter to use TypedEventEmitter (element-hq/element-web#29008). Contributed by @t3chguy.
  * Add `Recovery` section in the new user settings `Encryption` tab (element-hq/element-web#28673). Contributed by @florianduros.
  * Retry loading chunks to make the app more resilient (element-hq/element-web#29001). Contributed by @t3chguy.
  * Clear account idb table on logout (https://github.com/e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants