[gnome-shell] searchDisplay: Remove focus management code
- From: Jasper St. Pierre <jstpierre src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] searchDisplay: Remove focus management code
- Date: Wed, 29 May 2013 15:33:38 +0000 (UTC)
commit 32a53f7412ad4ebffe0b15ea886b0fa8aadb66c6
Author: Jasper St. Pierre <jstpierre mecheye net>
Date: Thu Feb 14 14:37:50 2013 -0500
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 files changed, 0 insertions(+), 9 deletions(-)
---
diff --git a/js/ui/searchDisplay.js b/js/ui/searchDisplay.js
index d7831a4..ebacdec 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();
}));
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]