From 32a53f7412ad4ebffe0b15ea886b0fa8aadb66c6 Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Thu, 14 Feb 2013 14:37:50 -0500 Subject: [PATCH] searchDisplay: Remove focus management code It turns out that this focus code broke sometime in the 3.6 cycle -- when updating results, the focus is always on the text entry, so this never gets called. We'll eventually replace it with something that keeps track of the focused result meta, but for now, remove it. https://bugzilla.gnome.org/show_bug.cgi?id=693836 --- js/ui/searchDisplay.js | 9 --------- 1 file changed, 9 deletions(-) diff --git a/js/ui/searchDisplay.js b/js/ui/searchDisplay.js index d7831a403..ebacdec2a 100644 --- a/js/ui/searchDisplay.js +++ b/js/ui/searchDisplay.js @@ -218,22 +218,13 @@ const SearchResultsBase = new Lang.Class({ this.provider.getResultMetas(results, Lang.bind(this, function(metas) { this.clear(); - // Hiding drops the key focus if we have it - let focus = global.stage.get_key_focus(); - // To avoid CSS transitions causing flickering when // the first search result stays the same, we hide the // content while filling in the results. this.actor.hide(); - this._renderResults(metas); - this._setMoreIconVisible(this.hasMoreResults() && this.provider.canLaunchSearch); - this.actor.show(); - if (this.actor.contains(focus)) - global.stage.set_key_focus(focus); - callback(); })); }