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

Add ability to customize room notification settings. #2887

Closed

Conversation

chrismoos
Copy link
Contributor

@chrismoos chrismoos commented Dec 22, 2019

Previously you could only toggle notifications to be "Mentions Only". Also, the UI was a bit confusing as it showed "Mute Notifications" which actually just set "Mentions Only", not "Mute".

This change let's you choose between:

  • All messages
  • Mentions only
  • Mute (no notifications)

notification_settings

Pull Request Checklist

  • UI change has been tested on both light and dark themes
  • Pull request is based on the develop branch
  • Pull request updates CHANGES.rst
  • Pull request includes screenshots or videos of UI changes
  • Pull request includes a sign off

Previously you could only toggle notifications to be "Mentions Only".
Also, the UI was a bit confusing as it showed "Mute Notifications" which
actually just set "Mentions Only", not "Mute".

This change let's you choose between:

* All messages
* Mentions only
* Mute (no notifications)

Signed-off-by: Chris Moos <[email protected]>
@chrismoos
Copy link
Contributor Author

Any update on this?

@OddHouseGames
Copy link

OddHouseGames commented Aug 29, 2020

Is there any update on this? The UI is very confusing here when trying to set to mentions only as all I can see is direct mentions toggle and mute. From a UX point of view it's not really clear for new user if direct messages toggle turns on ability to DM or means mentions only.image(see screenshot below)

@manuroe
Copy link
Member

manuroe commented Oct 1, 2021

We reviewed the design on all clients (android, ios and web) to have the same behavior that manages both e2ee and non e2ee rooms. See #4459.

@manuroe manuroe closed this Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants