From abdd8b330cd96f6c9fb7eb63ba5b17a8b71c164e Mon Sep 17 00:00:00 2001
From: Marina Zhurakhinskaya <marinaz@redhat.com>
Date: Thu, 1 Oct 2009 18:21:27 -0400
Subject: [PATCH] Don't show all sections each time the search is updated

We shouldn't show all sections each time the search is updated because that
breaks viewing search results for a single section and flickers headers for
sections with no results.
---
 js/ui/dash.js | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/js/ui/dash.js b/js/ui/dash.js
index b2686e564..28cd70065 100644
--- a/js/ui/dash.js
+++ b/js/ui/dash.js
@@ -889,6 +889,9 @@ Dash.prototype = {
     _updateDashActors: function() {
         if (this._searchPending) {
             this._searchResultsSection.actor.show();
+            // We initially hide all sections when we start a search. When the search timeout
+            // first runs, the sections that have matching results are shown. As the search
+            // is refined, only the sections that have matching results will be shown.
             for (let i = 0; i < this._searchSections.length; i++) {
                 let section = this._searchSections[i];
                 section.header.actor.hide();
@@ -897,13 +900,7 @@ Dash.prototype = {
             this._appsSection.actor.hide();
             this._placesSection.actor.hide();
             this._docsSection.actor.hide();
-        } else if (this._searchActive) {
-            for (let i = 0; i < this._searchSections.length; i++) {
-                let section = this._searchSections[i];
-                section.header.actor.show();
-                section.resultArea.actor.show();
-            }
-        } else {
+        } else if (!this._searchActive) {
             this._showAllSearchSections();
             this._searchResultsSection.actor.hide();
             this._appsSection.actor.show();