Bug 629438 - Add-ons manager add-on icon should be positioned correctly in list and not cause sizing differences. r+a=Mossop
authorMatthew Printz <hipokrit@hipokrit.com>
Thu, 03 Feb 2011 08:20:03 +0100
changeset 61832 a12d11c8912de1d650f8a587376fd03e4561f4db
parent 61831 2f353922a56cfd345ecf179e6c28a18517a39f88
child 61833 133443c095646cb8f83ef10f3b67090b9b80c764
push id18514
push userdgottwald@mozilla.com
push dateThu, 03 Feb 2011 07:24:03 +0000
treeherdermozilla-central@a12d11c8912d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs629438
milestone2.0b12pre
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 629438 - Add-ons manager add-on icon should be positioned correctly in list and not cause sizing differences. r+a=Mossop
toolkit/mozapps/extensions/content/extensions.xml
toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
toolkit/themes/pinstripe/mozapps/extensions/extensions.css
toolkit/themes/winstripe/mozapps/extensions/extensions.css
--- a/toolkit/mozapps/extensions/content/extensions.xml
+++ b/toolkit/mozapps/extensions/content/extensions.xml
@@ -816,18 +816,18 @@
                     oncommand="document.getBindingParent(this).restart();"/>
         <xul:button anonid="undo-btn" class="button-link"
                     label="&addon.undoAction.label;"
                     tooltipText="&addon.undoAction.tooltip;"
                     oncommand="document.getBindingParent(this).undo();"/>
         <xul:spacer flex="5000"/> <!-- Necessary to allow the message to wrap -->
       </xul:hbox>
 
-      <xul:hbox>
-        <xul:vbox align="center" pack="start" class="icon-container">
+      <xul:hbox align="start">
+        <xul:vbox align="center" pack="center" class="icon-container">
           <xul:image anonid="icon" class="icon"/>
         </xul:vbox>
         <xul:vbox flex="1">
           <xul:hbox align="start">
             <xul:vbox flex="1">
               <xul:hbox class="name-container" align="end">
                 <xul:label anonid="name" class="name" crop="end" flex="1"
                            xbl:inherits="value=name,tooltiptext=name"/>
--- a/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
@@ -415,17 +415,18 @@
 .details {
   cursor: pointer;
   margin: 0;
   -moz-margin-start: 10px;
 }
 
 .icon-container {
   width: 48px;
-  margin: 15px 7px 7px 7px;
+  height: 48px;
+  margin: 3px 7px;
 }
 
 .icon {
   list-style-image: url("chrome://mozapps/skin/extensions/extensionGeneric.png");
   max-width: 48px;
   max-height: 48px;
 }
 
--- a/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
@@ -441,17 +441,18 @@
 .details {
   cursor: pointer;
   margin: 0;
   -moz-margin-start: 10px;
 }
 
 .icon-container {
   width: 48px;
-  margin: 15px 7px 7px 7px;
+  height: 48px;
+  margin: 3px 7px;
 }
 
 .icon {
   list-style-image: url("chrome://mozapps/skin/extensions/extensionGeneric.png");
   max-width: 48px;
   max-height: 48px;
 }
 
--- a/toolkit/themes/winstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/winstripe/mozapps/extensions/extensions.css
@@ -514,17 +514,18 @@
 .details {
   cursor: pointer;
   margin: 0;
   -moz-margin-start: 10px;
 }
 
 .icon-container {
   width: 48px;
-  margin: 15px 7px 7px 7px;
+  height: 48px;
+  margin: 3px 7px;
 }
 
 .icon {
   list-style-image: url("chrome://mozapps/skin/extensions/extensionGeneric.png");
   max-width: 48px;
   max-height: 48px;
 }