From 30c6e541f15bb35d958c46939261672a0dca4f55 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Tue, 17 Feb 2015 03:40:25 +0100 Subject: [PATCH] autorunManager: Provide a createBanner() implementation Hotplug notifications use custom notification banners to include application icons in buttons. Bring those back by providing an appropriate createBanner() implementation. --- js/ui/components/autorunManager.js | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/js/ui/components/autorunManager.js b/js/ui/components/autorunManager.js index 59f5a048d..f8b79d89a 100644 --- a/js/ui/components/autorunManager.js +++ b/js/ui/components/autorunManager.js @@ -326,15 +326,21 @@ const AutorunNotification = new Lang.Class({ this._manager = manager; this._mount = source.mount; - source.apps.forEach(Lang.bind(this, function (app) { + // set the notification to urgent, so that it expands out + this.setUrgency(MessageTray.Urgency.CRITICAL); + }, + + createBanner: function() { + let banner = new MessageTray.NotificationBanner(this); + + this.source.apps.forEach(Lang.bind(this, function (app) { let actor = this._buttonForApp(app); if (actor) - this.addButton(actor); + banner.addButton(actor); })); - // set the notification to urgent, so that it expands out - this.setUrgency(MessageTray.Urgency.CRITICAL); + return banner; }, _buttonForApp: function(app) { @@ -352,6 +358,7 @@ const AutorunNotification = new Lang.Class({ let button = new St.Button({ child: box, x_fill: true, x_align: St.Align.START, + x_expand: true, button_mask: St.ButtonMask.ONE, style_class: 'hotplug-notification-item button' });