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 idunknown
push userunknown
push dateunknown
bugs629438
milestone2.0b12pre
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;
 }