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

Room Info: include also the current global/default notification mode #2965

Open
Velin92 opened this issue Dec 21, 2023 · 0 comments
Open

Room Info: include also the current global/default notification mode #2965

Velin92 opened this issue Dec 21, 2023 · 0 comments

Comments

@Velin92
Copy link
Contributor

Velin92 commented Dec 21, 2023

Proposal

So far the room info includes only the user defined notification mode, however it would be nice to also include a separate variable called default_notification_mode or something like that.

The thing is that we rely on the user defined one on Element X only to determine what settings badge should be displayed (for example we only rely on the user defined value to show the mute icon in the room list), however the green/grey dot state should instead depend on the current set notification mode, which should be the user defined if it exists, or the default one if a user one does not exist.

Alternatives

Another solution could be to have some kind of notification state variable that tells us through an enum:

  • if there is a notification with a mention
  • new notifications with no mention
  • new message but no notifications

And this variable evolves accordingly to the state of the current set notification mode for the room, be it user defined or not. However this could become very client specific so we could probably handle everything client side if we just have the default_notification_mode

@Velin92 Velin92 changed the title Room List API: Sync between devices the state of the mentioned indicator. Room Info: include also the current global/default notification mode Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant