From aa7ed319e96105984cac7d594d41ec1e35a7fee6 Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Fri, 21 Jun 2013 15:24:50 -0400 Subject: [PATCH] messageTray: Ensure notifications are focused after they are expanded If we focus notifications before they're expanded, the body and action area won't be visible, and the can_focus members like the text entry will not be able to be focused. Ensure that all of the all actors that would be in an expanded notification are visible before we attempt to focus them. https://bugzilla.gnome.org/show_bug.cgi?id=698778 --- js/ui/messageTray.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js index c0569247a..afbcf269a 100644 --- a/js/ui/messageTray.js +++ b/js/ui/messageTray.js @@ -2597,16 +2597,16 @@ const MessageTray = new Lang.Class({ }, _expandNotification: function(autoExpanding) { - // Don't focus notifications that are auto-expanding. - if (!autoExpanding) - this._ensureNotificationFocused(); - if (!this._notificationExpandedId) this._notificationExpandedId = this._notification.connect('expanded', Lang.bind(this, this._onNotificationExpanded)); // Don't animate changes in notifications that are auto-expanding. this._notification.expand(!autoExpanding); + + // Don't focus notifications that are auto-expanding. + if (!autoExpanding) + this._ensureNotificationFocused(); }, _onNotificationExpanded: function() {