From 69735940ec1519f5e9abbdcda6c43335cee69b86 Mon Sep 17 00:00:00 2001 From: Debarshi Ray Date: Sat, 1 Sep 2012 03:21:51 +0200 Subject: [PATCH] messageTray: Summary items should be square and have spacing Each summary icon is 48x48 with a padding of 6px on each side. Thus, each summary item is 60px wide. Therefore the summary mode should be 60px high instead of 72px. Changed the tray actor to use a ClutterBinLayout so that it honors the y-expand property of its children. https://bugzilla.gnome.org/show_bug.cgi?id=682248 --- data/theme/gnome-shell.css | 3 ++- js/ui/messageTray.js | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/data/theme/gnome-shell.css b/data/theme/gnome-shell.css index 5ce232b44..64e651af5 100644 --- a/data/theme/gnome-shell.css +++ b/data/theme/gnome-shell.css @@ -1418,7 +1418,8 @@ StButton.popup-menu-item:insensitive { #summary-mode { padding: 2px 0px 0px 4px; - height: 72px; + height: 60px; + spacing: 10px; } #summary-mode:rtl { diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js index 2d7ef8ea2..7ada61685 100644 --- a/js/ui/messageTray.js +++ b/js/ui/messageTray.js @@ -1361,6 +1361,7 @@ const MessageTray = new Lang.Class({ })); this.actor = new St.Widget({ name: 'message-tray', + layout_manager: new Clutter.BinLayout(), reactive: true, track_hover: true }); this.actor.connect('style-changed', Lang.bind(this, this._onStyleChanged)); @@ -1378,6 +1379,7 @@ const MessageTray = new Lang.Class({ this._notificationClickedId = 0; this._summaryBin = new St.Bin({ x_align: St.Align.END, + y_expand: true, // this is the Clutter property reactive: true }); this._summaryBin.connect('button-release-event', Lang.bind(this, function(actor, event) { this._setClickedSummaryItem(null);