From d684187f4294d876bf12244cd5a832221ac156dc Mon Sep 17 00:00:00 2001 From: Tulir Asokan Date: Mon, 26 Mar 2018 20:55:52 +0300 Subject: Don't send notifications for current room --- ui/view-main.go | 28 +++++++++++++++++----------- 1 file changed, 17 insertions(+), 11 deletions(-) (limited to 'ui') diff --git a/ui/view-main.go b/ui/view-main.go index a35c35e..fd05492 100644 --- a/ui/view-main.go +++ b/ui/view-main.go @@ -386,26 +386,32 @@ func sendNotification(room *rooms.Room, sender, text string, critical, sound boo } func (view *MainView) NotifyMessage(room *rooms.Room, message *types.Message, should pushrules.PushActionArrayShould) { + // Whether or not the room where the message came is the currently shown room. isCurrent := room.ID == view.CurrentRoomID() + // Whether or not the terminal window is focused. + isFocused := view.lastFocusTime.Add(30 * time.Second).Before(time.Now()) + + // Whether or not the push rules say this message should be notified about. + shouldNotify := (should.Notify || !should.NotifySpecified) && message.Sender != view.config.Session.UserID + if !isCurrent { + // The message is not in the current room, show new message status in room list. room.HasNewMessages = true + room.Highlighted = should.Highlight || room.Highlighted + if shouldNotify { + room.UnreadMessages++ + } } - shouldNotify := (should.Notify || !should.NotifySpecified) && message.Sender != view.config.Session.UserID - if shouldNotify { + + if shouldNotify && !isFocused { + // Push rules say notify and the terminal is not focused, send desktop notification. shouldPlaySound := should.PlaySound && should.SoundName == "default" sendNotification(room, message.Sender, message.Text, should.Highlight, shouldPlaySound) - if !isCurrent { - room.UnreadMessages++ - } } + if should.Highlight { + // Message is highlight, set color. message.TextColor = tcell.ColorYellow - if !isCurrent { - room.Highlighted = true - } - } - if should.PlaySound { - // TODO play sound } } -- cgit v1.2.3