From 31f1e9ff0a95be7d5fb4ffde6ecc6154b8ee4ca6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Tue, 18 Sep 2012 18:53:51 +0200 Subject: [PATCH] autorunManager: Do not mark the notification CRITICAL There is nothing particularly critical about this notification, it was only marked as such to get certain behavior like auto-expanding and sticking-around to be acknowledged by the user (as it offers more actions than the summary notification, so it is frustrating when it goes away because it was missed). As all notifications will now stay visible until we are sure the users has seen them, the latter reasoning no longer applies. Auto-expansion doesn't appear too important and may even be considered annoying by users, so remove the CRITICAL hint now. https://bugzilla.gnome.org/show_bug.cgi?id=657923 --- js/ui/components/autorunManager.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/js/ui/components/autorunManager.js b/js/ui/components/autorunManager.js index f8b79d89a..f800db885 100644 --- a/js/ui/components/autorunManager.js +++ b/js/ui/components/autorunManager.js @@ -325,9 +325,6 @@ const AutorunNotification = new Lang.Class({ this._manager = manager; this._mount = source.mount; - - // set the notification to urgent, so that it expands out - this.setUrgency(MessageTray.Urgency.CRITICAL); }, createBanner: function() {