Bug 891056 (part 1) - Code cleanup for autocomplete CSS [r=rsilveira]
authorMatt Brubeck <mbrubeck@mozilla.com>
Thu, 05 Sep 2013 17:40:17 -0700
changeset 158809 c3b4537fb2287290966e2908ecee801cd241514e
parent 158808 3194bc73870e1467a4e5eb01dd082349b3f7c9cc
child 158810 b12a19b0e5008f630d3dc30c9488f461ab0187c0
push id2961
push userlsblakk@mozilla.com
push dateMon, 28 Oct 2013 21:59:28 +0000
treeherdermozilla-beta@73ef4f13486f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrsilveira
bugs891056
milestone26.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 891056 (part 1) - Code cleanup for autocomplete CSS [r=rsilveira]
browser/metro/base/content/bindings/urlbar.xml
browser/metro/base/content/browser-ui.js
browser/metro/theme/browser.css
--- a/browser/metro/base/content/bindings/urlbar.xml
+++ b/browser/metro/base/content/bindings/urlbar.xml
@@ -480,17 +480,17 @@
           if (this.submitURL(event))
             return;
         ]]>
       </handler>
     </handlers>
   </binding>
 
   <binding id="urlbar-autocomplete">
-    <content orient="horizontal">
+    <content>
       <xul:vbox id="results-vbox" anonid="results-container" flex="1">
         <xul:label class="meta-section-title"
                    value="&autocompleteResultsHeader.label;"/>
         <richgrid id="results-richgrid" anonid="results" rows="3" flex="1"
                   seltype="single" deferlayout="true"/>
       </xul:vbox>
 
       <xul:vbox id="searches-vbox" flex="1">
--- a/browser/metro/base/content/browser-ui.js
+++ b/browser/metro/base/content/browser-ui.js
@@ -587,20 +587,16 @@ var BrowserUI = {
             this.changeDebugPort(Services.prefs.getIntPref(aData));
             break;
         }
         break;
       case "metro_viewstate_changed":
         this._adjustDOMforViewState(aData);
         if (aData == "snapped") {
           FlyoutPanelsUI.hide();
-          Elements.autocomplete.setAttribute("orient", "vertical");
-        }
-        else {
-          Elements.autocomplete.setAttribute("orient", "horizontal");
         }
 
         break;
     }
   },
 
   /*********************************
    * Internal utils
--- a/browser/metro/theme/browser.css
+++ b/browser/metro/theme/browser.css
@@ -209,28 +209,24 @@ documenttab[selected] .documenttab-selec
   /* allot space for 3 tile columns for the topsites grid */
   min-width: calc(3 * @grid_double_column_width@);
 }
 
 #content-viewport[startpage] browser {
   padding-bottom: @toolbar_height@;
 }
 
+#urlbar-autocomplete[viewstate="snapped"],
+#urlbar-autocomplete[viewstate="portrait"],
 #start-container[viewstate="snapped"],
 #start-container[viewstate="portrait"] {
   -moz-box-orient: vertical;
 }
 
-/*Formatting for the limited horizontal space of snapped*/
-#start-autocomplete[viewstate="snapped"] .richgrid-item-content {
-  -moz-box-orient: horizontal;
-}
-
-#start-container,
-#start-autocomplete {
+#start-container {
   padding-left: 0;
   padding-right: 0;
 }
 
 #start-container[viewstate="snapped"] > .meta-section {
   margin: 0 @metro_spacing_xnormal@;
   min-width: @grid_double_column_width@;
 }
@@ -1293,9 +1289,9 @@ setting[type="radio"] > vbox {
 #clearprivacythrobber .progressBall {
   margin: 2px;
   width: 22px;
   height: 22px;
 }
 
 #clear-notification-done {
   font-weight: bold;
-}
\ No newline at end of file
+}