Qabel / qabel-desktop

(B2C) :computer: Desktop frontend of Qabel
Other
11 stars 12 forks source link

Do not show every received message as a notification #527

Open audax opened 7 years ago

audax commented 7 years ago

When I received multiple messages from contacts while I was offline, I receive a tray notification for every single message.

Only one notification should be shown (maybe on for each identity that received new messages) which shows a summary of the messages, for example:

"5 new messages from ContactA, ContactB, ContactC"

See the summary headline of https://material.google.com/patterns/notifications.html#notifications-usage

julianseeger commented 7 years ago

What did it look like? Did they stack (hiding each other / on top of each other)? Or did the second appear when the first vanished? And..does it already work on android?

audax commented 7 years ago

They appeared one after another, which took quite a while. It is already implemented on android like that.

The code for combining the notifications is here: https://github.com/Qabel/qabel-android/blob/master/qabelbox/src/main/java/de/qabel/qabelbox/chat/notifications/MainChatNotificationManager.kt#L78

julianseeger commented 7 years ago

TBD if the messages should only be combined if they are already there or also if they arrive in quick succession...