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

Notifications experience is not the same when compared to the web / IOS client #4632

Closed
alongls opened this issue Dec 5, 2021 · 12 comments
Closed
Assignees
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Team: Delight Z-parity-with-web Z-WTF WTF moment: High Impact, Low Effort

Comments

@alongls
Copy link
Contributor

alongls commented Dec 5, 2021

Steps to reproduce

  1. Where are you starting? What can you see?
    when testing the notifications when message is received we see that the android client behavior is different than the IOS or WEB client.
    when several messages from the same person are received in IOS device we hear notification sound per message that is received (e.g. if received 5 messages from the same person we will hear 5 beeps) also the app indicator will show 5 to let us know that 5 messages are waiting for us
    in the web client we will also hear sounds as per message that is received.
    in the Android client it is not like that
    the Android client will play one sound even if multiple message are received from the same person
    the android client will show on the app icon that 1 message is waiting for us although there are multiple messages waiting

  2. What do you click?

  3. More steps…

Outcome

What did you expect?

sound per received message even if messages are received from the same person / room (as in the IOS / WEC client and as in whatsapp)

What happened instead?

one sound although several messages were received. number on icon was also wrong

Your phone model

Samsung s21 (several model tested)

Operating system version

android 7-10

Application version and app store

element version 1.3.7 olm 3.2.4

Homeserver

matrix.org

Will you send logs?

Yes

@alongls alongls added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Dec 5, 2021
@ouchadam ouchadam added A-Notifications X-Needs-Product Issue needs input from Product team Z-parity-with-web labels Dec 7, 2021
@ouchadam
Copy link
Contributor

ouchadam commented Dec 7, 2021

I'm tempted to swap this to an enhancement, previously we've had reports of the notifications happening too often, especially in high traffic rooms

This feels like direct messages could have different behaviour to rooms or we allow the mobile clients to control noisy notifications cc @daniellekirkwood

@ouchadam ouchadam added O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist labels Dec 7, 2021
@alongls
Copy link
Contributor Author

alongls commented Dec 8, 2021

Hello
what ever you decide to do you need to make sure that the behavior will be the same across all platforms.
you can not maintain 3 platforms (web, ios, android) that behave differently since it will confuse the users and will guide them away from this solution.

@daniellekirkwood
Copy link
Contributor

Great issue to raise - and great context added, thanks both.

I agree with @ouchadam that this is an Enhancement for us, however I think you raise a great point @alongls and we should look at the solution holistically across platforms.

@daniellekirkwood daniellekirkwood added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements and removed T-Defect Something isn't working: bugs, crashes, hangs and other reported problems labels Dec 10, 2021
@binarydad
Copy link

I was wondering this too. I talk to my son on Element (android) and he only hears my first notification, until he either reads it or swipes it away. If that first notification isn't heard, it's not obvious that a new message has been received.

@alongls
Copy link
Contributor Author

alongls commented Jan 19, 2022

fully agree
this is why we moved away from the app until this issue will be resolved (if the team will decide to change the current behavior).

@matrixes
Copy link

I miss a lot of conversations on specifically Android that I don't on either web or IOS, because of the way notifications behave on Android.

As others have stated, I agree the behaviour really needs to be consistent over the different platforms because as it is now, (as Android is the only client behaving like this and it being my daily driver), I have to double and triple check my phone to see if I've missed out on any conversation.

If there's any suggestion gathering from users at this point, I'd like to propose that

  • all platforms behave the same (as their defaults)
  • giving the end users more control of how they'd like to be notified is preffered
  • to preserve consistency between platforms, even if users are able to affect settings much more granularly by themselves, the app could be set to the same behaviour as the other platforms by default on first install.

I'm tempted to swap this to an enhancement, previously we've had reports of the notifications happening too often, especially in high traffic rooms

This makes me curious. Is it only/majority Android-users that experience that notifications happen too often? Considering that web and IOS get notified for each message (unless muted, of course), wouldn't that also result in the same type of reports from other platform users?
Or is this perhaps a result of a higher number of Android users?
Anyways, it was just something that stood out that I became curious of.

This feels like direct messages could have different behaviour to rooms or we allow the mobile clients to control noisy notifications cc @daniellekirkwood

As a sidenote; I have also had a request up for being able to set individual sounds for individual rooms for Android since a few years back. It would give me freedom to, by sound, determine if I'd like to get involved in the discussions (#534)

@giomfo
Copy link
Member

giomfo commented Mar 29, 2022

@daniellekirkwood I would suggest notifying for each message on Android (when the notifications are enabled for the room) to adopt the same behavior as iOS and Web, until we rework/enhance the notifications handling

If you agree, we can prepare a PR for this change. Please let me know if you prefer keeping it like this

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Mar 30, 2022

Yes @giomfo, let's complete the work to make it consistent across platforms. Then we can work on improving it.

As it sounds like the work needed for this issue has been selected, I'm removing the Needs-Product and my own assignment.

@alongls
Copy link
Contributor Author

alongls commented Apr 7, 2022

Great news
waiting for the app update.
thanks

@lindhe
Copy link
Contributor

lindhe commented May 18, 2022

I believe release v1.4.16 tried to reference this issue, but used the incorrect issue number.

  • Notify the user for each new message (#46312)

@binarydad
Copy link

FYI, it seems there is still an issue with older Android devices (ver 8 and older, IIRC) that are making TWO sounds for the first message received, and one for each one thereafter.

I noticed the same behavior when I was messing with tackling this same issue. I'll submit a bug later when I get around to it unless someone else does first.

@Claire1817
Copy link
Contributor

Thanks @binarydad for your feedback, can you create a new issue please ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Team: Delight Z-parity-with-web Z-WTF WTF moment: High Impact, Low Effort
Projects
None yet
Development

No branches or pull requests

9 participants