Element-web: Notification sound plays even when active in the conversation
Description
When in a conversation / 1 to 1 room, the notification sound plays on every received message even if i the tab is in focus and you are active in that conversation.
Steps to reproduce
- Set notifications to "noisy" on "Messages in one-to-one chats" and on "Messages containing my name".
- Start chatting with someone in a one-to-one chat or be mentioned in a bigger room.
- Stay in that window/tab/room.
- Notice the notification sound playing on every message received.
Expectation would be that sound doesn't play when you are active in the conversation.
If notifications are set to anything but noisy, no sound is played at all, regardless of the window being in focus or not.
Version information
Platform: Desktop
OS: Ubuntu Linux 16.04
- Version: 0.9.7
mvintila
All 15 comments
Flinging this question at @dbkr - do you know whether this is the designed behaviour for noisy notifications?
It certainly sounds... noisy :P
lampholder
on 16 Mar 2017
i think this is behaving as expected based on reading https://github.com/matrix-org/matrix-react-sdk/blob/develop/src/Notifier.js#L224, but it's dumb - we should be surely be suppressing for the room which is active in the foreground (assuming the user isn't idle).
ara4n
on 1 Apr 2017
Agreed we should probably suppress notification sounds for the room being viewed when the app is in the foreground and the user isn't idle. Blocker on this is that Notifier doesn't know what room is currently visible.
dbkr
on 8 May 2017
In my opinion it doesn't really matter which room is in focus, if the app is in focus notification sounds should be disabled.
The purpose of the sound notifications is to let you know that you have messages when away from your device. I would be ok to even mute notifications when you are active on the device (computer not idle, phone screen on, etc).
If the app is in focus i can see the the number of new messages in the left panel (or notification bar on the phone), no need to play sound.
mvintila
on 8 May 2017
This should be much easier now with the RVS
t3chguy
on 17 Jul 2017
any update on this? No clue what RVS is, but these notifications are getting pretty annoying.
mickeyvanolst
on 10 Oct 2017
RVS=RoomViewStore
and no one is working on this = no update
@ara4n / @dbkr what do you feel the behaviour should be, if window=active && room=active silence the notif, if so I'll get on it
t3chguy
on 10 Oct 2017
personally I don't think it should have anything to do with rooms, if you're using the desktop app of riot, your phone shouldn't be vibrating at any time, its pretty easy to see the notifications in the riot desktop app at that point.
mickeyvanolst
on 10 Oct 2017
@popsure that's a completely different issue
the issue you want is https://github.com/vector-im/riot-web/issues/469
t3chguy
on 10 Oct 2017
@t3chguy thanks for clarifying, I got redirected to here from that thread, my mistake.
mickeyvanolst
on 10 Oct 2017
:musical_note: PLING! :musical_note:
AlexDaniel
on 31 Mar 2018
This is very annoying for me. Is any fix in preparation for that?
florealcab
on 30 Nov 2019
I am still experiencing this 3 years later, I take it not much has happened since.
squidink7
on 27 Mar 2020
Pidgin IM used to have "message starts conversation" and "when in focus" options, which I strongly think should be copied in Riot.
geckolinux
on 11 Apr 2020
Same here :(
henricazottes
on 17 Jun 2020
Related issues
bagage
·
3Comments
niedzielski
·
3Comments
turt2live
·
3Comments
lampholder
·
3Comments
MurzNN
·
3Comments
Most helpful comment
I am still experiencing this 3 years later, I take it not much has happened since.