Bug 528279: Portion of larry ui shown when adding a search engine [r=vingtetun]
authorMark Finkle <mfinkle@mozilla.com>
Fri, 20 Nov 2009 23:41:58 -0500
changeset 65829 bc5c81ff617823247374cb3a7be48eab5c34c8c9
parent 65828 f0dd7adbf05a27aa71df212795246fa6da0c2331
child 65830 39a081d00edc0060b2ae6f16d4915667402c9df4
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvingtetun
bugs528279
Bug 528279: Portion of larry ui shown when adding a search engine [r=vingtetun]
mobile/chrome/content/browser.js
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -1472,35 +1472,35 @@ const BrowserSearch = {
     }, this);
 
     let container = document.getElementById('search-container');
     let buttons = container.getElementsByAttribute("class", "search-engine-button button-dark");
     for (let i=0; i<buttons.length; i++)
       container.removeChild(buttons[i]);
 
     if (newEngines.length == 0) {
-      container.hidden = true;
+      container.collapsed = true;
       return;
     }
 
     // XXX limit to the first search engine for now
     for (let i = 0; i<1; i++) {
       let button = document.createElement("button");
       button.className = "search-engine-button button-dark";
-      button.setAttribute("oncommand", "BrowserSearch.addPermanentSearchEngine(this.engine);this.parentNode.hidden=true;");
+      button.setAttribute("oncommand", "BrowserSearch.addPermanentSearchEngine(this.engine);this.parentNode.collapsed=true;");
       
       let engine = newEngines[i];
       button.engine = engine.engine;
       button.setAttribute("label", engine.engine.title);
       button.setAttribute("image", BrowserUI._favicon.src);
 
       container.appendChild(button);
     }
 
-    container.hidden = false;
+    container.collapsed = false;
   },
 
   addPermanentSearchEngine: function (aEngine) {
     let iconURL = BrowserUI._favicon.src;
     this.searchService.addEngine(aEngine.href, Ci.nsISearchEngine.DATA_XML, iconURL, false);
 
     this._engines = null;
   },