Don't display notifications for open windows.
Just update and draw attention to them. Fixes #238 // FREEBIE
This commit is contained in:
		
					parent
					
						
							
								d8f17f9631
							
						
					
				
			
			
				commit
				
					
						3eafefe18e
					
				
			
		
					 2 changed files with 10 additions and 3 deletions
				
			
		| 
						 | 
				
			
			@ -48,7 +48,13 @@
 | 
			
		|||
    };
 | 
			
		||||
 | 
			
		||||
    window.notifyConversation = function(message) {
 | 
			
		||||
        if (Whisper.Notifications.isEnabled()) {
 | 
			
		||||
        var conversationId = message.get('conversationId');
 | 
			
		||||
        var windowId = windowMap.windowIdFrom(conversationId);
 | 
			
		||||
        if (windowId) {
 | 
			
		||||
            // already open
 | 
			
		||||
            updateConversation(conversationId);
 | 
			
		||||
            extension.windows.drawAttention(windowId);
 | 
			
		||||
        } else if (Whisper.Notifications.isEnabled()) {
 | 
			
		||||
            var conversation = getConversation({id: message.get('conversationId')});
 | 
			
		||||
            var sender = getConversation({id: message.get('source')});
 | 
			
		||||
            conversation.fetch().then(function() {
 | 
			
		||||
| 
						 | 
				
			
			@ -65,7 +71,6 @@
 | 
			
		|||
            });
 | 
			
		||||
            conversation.fetchMessages();
 | 
			
		||||
        } else {
 | 
			
		||||
            var conversationId = message.get('conversationId');
 | 
			
		||||
            openConversation(conversationId);
 | 
			
		||||
            var windowId = windowMap.windowIdFrom(conversationId);
 | 
			
		||||
            extension.windows.drawAttention(windowId);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue