Bug 611861 - add-ons list cramped compared to awesomescreen [r=vingtetun]
authorMark Finkle <mfinkle@mozilla.com>
Wed, 22 Dec 2010 15:13:39 -0500
changeset 67158 040fc388b73af793c7fb7e1cd582951b575a5951
parent 67157 be9046794ff37d8979cdc195357c9f8d3c1adddf
child 67159 5b544d1ed1126c6fba82de36a89a11ec22f7e5dd
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
bugs611861
Bug 611861 - add-ons list cramped compared to awesomescreen [r=vingtetun]
mobile/chrome/content/bindings/extensions.xml
mobile/themes/core/browser.css
--- a/mobile/chrome/content/bindings/extensions.xml
+++ b/mobile/chrome/content/bindings/extensions.xml
@@ -130,17 +130,17 @@
         </body>
       </method>
     </implementation>
   </binding>
 
   <binding id="extension-searchplugin" extends="chrome://browser/content/bindings.xml#richlistitem">
     <content orient="vertical">
       <xul:hbox align="start">
-        <xul:image width="32" height="32" xbl:inherits="src=iconURL"/>
+        <xul:image class="searchplugin-image" xbl:inherits="src=iconURL"/>
         <xul:vbox flex="1">
           <xul:hbox align="center">
             <xul:label class="title" xbl:inherits="value=name" crop="end" flex="1"/>
             <xul:spacer flex="1000"/>
             <xul:label class="normal" xbl:inherits="value=typeLabel"/>
           </xul:hbox>
           <xul:vbox>
             <xul:label class="normal hide-on-select" xbl:inherits="value=description" crop="end" flex="1"/>
@@ -179,17 +179,17 @@
         <![CDATA[
           if (!this.hasAttribute("homepageURL"))
             this._showPageButton.setAttribute("disabled", "true");
         ]]>
       </constructor>
     </implementation>
     <content orient="vertical">
       <xul:hbox align="start">
-        <xul:image xbl:inherits="src=iconURL"/>
+        <xul:image class="addon-image" xbl:inherits="src=iconURL"/>
         <xul:vbox flex="1">
           <xul:hbox align="center">
             <xul:label class="title" xbl:inherits="value=name" crop="end" flex="1"/>
             <xul:label class="normal" xbl:inherits="value=version"/>
             <xul:spacer flex="1000"/>
             <xul:hbox class="addon-type-or-rating" align="center" xbl:inherits="rating"/>
           </xul:hbox>
           <xul:vbox>
--- a/mobile/themes/core/browser.css
+++ b/mobile/themes/core/browser.css
@@ -473,20 +473,23 @@ toolbarbutton.choice-remotetabs {
 
 %expand @media (__ORIENTATION__: portrait) {
   #addons-repo {
     -moz-box-orient: vertical;
     -moz-box-pack: center;
   }
 }
 
-.addon-image {
+.addon-image,
+.searchplugin-image {
   list-style-image: url("chrome://mozapps/skin/xpinstall/xpinstallItemGeneric.png");
   width: 32px;
   height: 32px;
+  -moz-margin-start: 8px;
+  -moz-margin-end: 8px;
 }
 
 .addon-rating[rating] {
   width: 78px;
   height: 18px;
   list-style-image: url("chrome://browser/skin/images/ratings-18.png");
 }
 
@@ -509,26 +512,23 @@ toolbarbutton.choice-remotetabs {
 .addon-rating[rating="4"] {
   -moz-image-region: rect(72px 78px 90px 0px);
 }
 
 .addon-rating[rating="5"] {
   -moz-image-region: rect(90px 78px 108px 0px);
 }
 
-.addon-showmore-image {
-  width: 32px;
-  height: 32px;
-  list-style-image: url("chrome://browser/skin/images/addons-32.png");
-}
-
+.addon-showmore-image,
 .addon-banner-image {
   width: 32px;
   height: 32px;
   list-style-image: url("chrome://browser/skin/images/addons-32.png");
+  -moz-margin-start: 8px;
+  -moz-margin-end: 8px;
 }
 
 richlistitem[isDisabled="true"] .title {
   color: gray;
 }
 
 richlistitem[isDisabled="true"] .normal {
   color: lightgray;