-
Notifications
You must be signed in to change notification settings - Fork 12
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 keyword mentions as an option to the notif panel #1212
Comments
We also aren't highlighting our userid/displayname notifications as the zeplin design shows. |
👍 It's particularly weird when I glimpse a transient notification (with the Riot window not in front) not long enough to know the room where a word was mentioned. Sometimes if I'm lucky there'll be a corresponding notification on Android. In these cases I can swipe down and refrain from tapping, the notification allows me to know the word and the room. |
I think I finally figured out the distinction on how to make this work thanks to the details explained by @clokep in matrix-org/synapse#15140 (comment). Also tested and it does behave this way! The notification panel only shows Confusingly, the In order to make this work, you must set We now have a workaround but this doesn't take away from the fact that Element notification settings are crazy obtuse. I would expect Please note that messages in encrypted rooms won't show up at all. This is tracked separately by element-hq/element-web#6874 |
@daniellekirkwood this might need considering as part of GYU as we'll be removing the On/Noisy differentiation so keywords might fall off the notification panel |
Thanks @t3chguy We're not going to consider this as a part of the GYU piece of work but we are going to ensure that this notifications panel is in our backlog/roadmap. As part of the work we're doing relating to Design Systems we'll be moving the access point to this panel and highlighting to users that we recognise it needs improvement. At some point in 2024 we should address the panel holistically and ensure it works for the use cases (ideally including encrypted rooms too) |
The text was updated successfully, but these errors were encountered: