[gnome-shell] messageTray: Correctly show the icon on a notification stack
- From: Jasper St. Pierre <jstpierre src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] messageTray: Correctly show the icon on a notification stack
- Date: Fri, 17 Feb 2012 19:05:42 +0000 (UTC)
commit fd99d13f04b4618462a218d1476f820f182626e5
Author: Jasper St. Pierre <jstpierre mecheye net>
Date: Thu Feb 16 17:23:16 2012 -0500
messageTray: Correctly show the icon on a notification stack
An actor is removed from its parent after it emits the destroy signal,
so we can't just check if the notification stack has more than one
notification -- we need to check if there's a notification there
that is not the current one.
This was causing spew in the form of:
"this.notificationStack.get_children()[0]._delegate.setIconVisible
is not a function"
js/ui/messageTray.js | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 1c7f0ba..9ccc6e4 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1327,8 +1327,9 @@ const SummaryItem = new Lang.Class({
}
}
- if (this.notificationStack.get_children().length > 0)
- this.notificationStack.get_children()[0]._delegate.setIconVisible(true);
+ let firstNotification = this._stackedNotifications[0];
+ if (firstNotification)
+ firstNotification.notification.setIconVisible(true);
}
});
Signals.addSignalMethods(SummaryItem.prototype);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]