[gnome-shell/gnome-3-36] messageTray: Don't play sound when banners are disabled



commit 8c19cd528ab03abf18dd7958807ebf3630982395
Author: Florian Müllner <fmuellner gnome org>
Date:   Tue Oct 27 01:03:43 2020 +0100

    messageTray: Don't play sound when banners are disabled
    
    When support for notification sounds was added, it made some sense
    to keep the 'enable-sound' setting independent from 'show-banners'.
    
    However that changed when 'show-banners' was rebranded in the UI as
    "Do Not Disturb", as sounds are at least as disturbing as the banners.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/2873
    
    Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1814>

 js/ui/messageTray.js | 2 --
 1 file changed, 2 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 970d2dd9dc..8f49ffcd61 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -855,8 +855,6 @@ var Source = GObject.registerClass({
 
         if (this.policy.showBanners || notification.urgency == Urgency.CRITICAL)
             this.emit('notification-show', notification);
-        else
-            notification.playSound();
     }
 
     notify(propName) {


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]