From c2f304f3bc846f0643c6a3e063a0102f89cf3fa7 Mon Sep 17 00:00:00 2001 From: Rui Matos Date: Sat, 10 Mar 2012 16:20:47 +0100 Subject: [PATCH] searchDisplay: Check the provider container for visibility It's the provider container that might be !visible, not the individual results. --- js/ui/searchDisplay.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/js/ui/searchDisplay.js b/js/ui/searchDisplay.js index e6bff4180..07984a479 100644 --- a/js/ui/searchDisplay.js +++ b/js/ui/searchDisplay.js @@ -348,8 +348,11 @@ const SearchResults = new Lang.Class({ if (meta.hasPendingResults) return; + if (!meta.actor.visible) + continue; + let firstResult = meta.resultDisplay.getFirstResult(); - if (firstResult && firstResult.actor.visible) { + if (firstResult) { newDefaultResult = firstResult; break; // select this one! } @@ -436,7 +439,7 @@ const SearchResults = new Lang.Class({ }, activateDefault: function() { - if (this._defaultResult && this._defaultResult.actor.visible) + if (this._defaultResult) this._defaultResult.activate(); },