[gnome-shell/wip/fmuellner/notification-redux+sass: 133/141] messageTray: Limit number of buttons in banners



commit 746fc95bb012b971dbcbb1956722c18e2c2e0d18
Author: Florian Müllner <fmuellner gnome org>
Date:   Tue Feb 17 03:53:00 2015 +0100

    messageTray: Limit number of buttons in banners
    
    The design states that notifications should have a maximum of three
    buttons, so enforce this in the new banner actors.

 js/ui/messageTray.js |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 32f8578..b8b45ff 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -30,6 +30,7 @@ const LONGER_HIDE_TIMEOUT = 0.6;
 
 const MAX_NOTIFICATIONS_IN_QUEUE = 3;
 const MAX_NOTIFICATIONS_PER_SOURCE = 3;
+const MAX_NOTIFICATION_BUTTONS = 3;
 
 // We delay hiding of the tray if the mouse is within MOUSE_LEFT_ACTOR_THRESHOLD
 // range from the point where it left the tray.
@@ -1170,6 +1171,9 @@ const NotificationBanner = new Lang.Class({
             global.focus_manager.add_group(this._buttonBox);
         }
 
+        if (this._buttonBox.get_n_children() >= MAX_NOTIFICATION_BUTTONS)
+            return null;
+
         this._buttonBox.add(button);
         button.connect('clicked', Lang.bind(this, function() {
             callback();


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