-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Update room header icon for world_readable rooms #31915
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
Conversation
Followup on #31879: change the icon shown for world_readable rooms.
| "room_is_public": "This room is public", | ||
| "shared_history_tooltip": "New members see history" | ||
| "shared_history_tooltip": "New members see history", | ||
| "world_readable_history_tooltip": "Anyone can see history" |
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.
How is this true? Unless it really means Anyone can see [undecryptable] history
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.
See the explanation in the description. Specifically, @mxandreas said:
I would keep it simple and always communicate just the history visibility setting. Then, yes, there are cases where somebody can't actually see the history - because they did not join via invite, because they did not have keys because they were not part of room, etc. It is not feasible to put all of that on UI and expect users to understand that. The next best approach is that we have a FAQ that explain those cases if some of them become more popular.
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.
Having discussed this with the tea
/me pictures vdh talking to his pg tips
Followup on element-hq#31879: change the icon shown for world_readable rooms.
…880) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [element-hq/element-web](https://github.com/element-hq/element-web) | patch | `v1.12.9` → `v1.12.10` | --- ### Release Notes <details> <summary>element-hq/element-web (element-hq/element-web)</summary> ### [`v1.12.10`](https://github.com/element-hq/element-web/releases/tag/v1.12.10) [Compare Source](element-hq/element-web@v1.12.9...v1.12.10) #### ✨ Features - Support additional\_creators in /upgraderoom (MSC4289) ([#​31934](element-hq/element-web#31934)). Contributed by [@​andybalaam](https://github.com/andybalaam). - Update room header icon for world\_readable rooms ([#​31915](element-hq/element-web#31915)). Contributed by [@​richvdh](https://github.com/richvdh). - Show an icon in the room header for shared history ([#​31879](element-hq/element-web#31879)). Contributed by [@​richvdh](https://github.com/richvdh). - Remove "history may be shared" banner. ([#​31881](element-hq/element-web#31881)). Contributed by [@​kaylendog](https://github.com/kaylendog). - Allow dismissing 'Key storage out of sync' temporarily ([#​31455](element-hq/element-web#31455)). Contributed by [@​andybalaam](https://github.com/andybalaam). - Add `resolutions` entry for `matrix-widget-api` to package.json ([#​31851](element-hq/element-web#31851)). Contributed by [@​toger5](https://github.com/toger5). - Improve visibility under contrast control mode ([#​31847](element-hq/element-web#31847)). Contributed by [@​t3chguy](https://github.com/t3chguy). - Unread Sorting - Add option for sorting in `OptionsMenuView` ([#​31754](element-hq/element-web#31754)). Contributed by [@​MidhunSureshR](https://github.com/MidhunSureshR). - Unread sorting - Implement sorter and use it in the room list store ([#​31723](element-hq/element-web#31723)). Contributed by [@​MidhunSureshR](https://github.com/MidhunSureshR). - Allow Element Call widgets to receive sticky events ([#​31843](element-hq/element-web#31843)). Contributed by [@​robintown](https://github.com/robintown). - Improve icon rendering accessibility ([#​31791](element-hq/element-web#31791)). Contributed by [@​t3chguy](https://github.com/t3chguy). - Add message preview toggle to room list header option ([#​31821](element-hq/element-web#31821)). Contributed by [@​florianduros](https://github.com/florianduros). #### 🐛 Bug Fixes - \[Backport staging] Fix room list not being cleared ([#​32438](element-hq/element-web#32438)). Contributed by [@​RiotRobot](https://github.com/RiotRobot). - Fix failure to update room info panel on joinrule change ([#​31938](element-hq/element-web#31938)). Contributed by [@​richvdh](https://github.com/richvdh). - Throttle space notification state calculation ([#​31922](element-hq/element-web#31922)). Contributed by [@​dbkr](https://github.com/dbkr). - Fix emoji verification responsive layout ([#​31899](element-hq/element-web#31899)). Contributed by [@​t3chguy](https://github.com/t3chguy). - Add patch for linkify to fix doctype handling ([#​31900](element-hq/element-web#31900)). Contributed by [@​dbkr](https://github.com/dbkr). - Fix rooms with no messages appearing at the top of the room list ([#​31798](element-hq/element-web#31798)). Contributed by [@​MidhunSureshR](https://github.com/MidhunSureshR). - Fix room list menu flashes when menu is closed ([#​31868](element-hq/element-web#31868)). Contributed by [@​florianduros](https://github.com/florianduros). - Message preview toggle is inverted in room list header ([#​31865](element-hq/element-web#31865)). Contributed by [@​florianduros](https://github.com/florianduros). - Fix duplicate toasts appearing for the same call if two events appear. ([#​31693](element-hq/element-web#31693)). Contributed by [@​Half-Shot](https://github.com/Half-Shot). - Fix ability to send rageshake during session restore failure ([#​31848](element-hq/element-web#31848)). Contributed by [@​t3chguy](https://github.com/t3chguy). - Fix mis-alignment of `Threads` right panel title ([#​31849](element-hq/element-web#31849)). Contributed by [@​t3chguy](https://github.com/t3chguy). - Unset buttons does not include color inherit ([#​31801](element-hq/element-web#31801)). Contributed by [@​Philldomd](https://github.com/Philldomd). </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4zLjYiLCJ1cGRhdGVkSW5WZXIiOiI0My4zLjYiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbImF1dG9tZXJnZSIsImltYWdlIl19--> Reviewed-on: https://gitea.alexlebens.dev/alexlebens/infrastructure/pulls/3880 Co-authored-by: Renovate Bot <[email protected]> Co-committed-by: Renovate Bot <[email protected]>
Followup on #31879: change the icon shown for world_readable rooms.
Having discussed this with the tea, we want to show a different icon (with different text) for rooms where the history_visibility is
world_readablerather thanshared.We considered whether it was accurate to say "Anyone can read history" when you actually need an invite for that to be the case, but decided that explaining all the nuances in the UI would not be feasible, and it was better just to reflect the history visibility settings.
Fixes: #31858 (again)
Screenshot: