From c89af0cea430d4c5652d314d5aa0ee0937a41400 Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Tue, 3 Dec 2013 19:21:08 -0500 Subject: [PATCH] messageTray: Only attempt to grab the summary box pointer after showing it For mysterious reasons I'm not sure of myself, navigate_focus will only focus mapped actors. So, make sure the widget is showing before navigating to it. https://bugzilla.gnome.org/show_bug.cgi?id=709853 --- js/ui/messageTray.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js index aa93640a6..4372c36f1 100644 --- a/js/ui/messageTray.js +++ b/js/ui/messageTray.js @@ -2809,13 +2809,13 @@ const MessageTray = new Lang.Class({ Lang.bind(this, this._onSourceDoneDisplayingContent)); this._summaryBoxPointer.bin.child = child; - this._grabHelper.grab({ actor: this._summaryBoxPointer.bin.child, - onUngrab: Lang.bind(this, this._onSummaryBoxPointerUngrabbed) }); - this._summaryBoxPointer.actor.opacity = 0; this._summaryBoxPointer.actor.show(); this._adjustSummaryBoxPointerPosition(); + this._grabHelper.grab({ actor: this._summaryBoxPointer.bin.child, + onUngrab: Lang.bind(this, this._onSummaryBoxPointerUngrabbed) }); + this._summaryBoxPointerState = State.SHOWING; this._summaryBoxPointer.show(BoxPointer.PopupAnimation.FULL, Lang.bind(this, function() { this._summaryBoxPointerState = State.SHOWN;