Bug 715735 - Detail view icon has no dimensions set, causes a relayout when it loads; r=Unfocused
authorGeoff Lankow <geoff@darktrojan.net>
Tue, 10 Jan 2012 20:15:51 +1300
changeset 85552 4c0a14d58397f07543c80262ccd81319e1807fe7
parent 85551 a9eb2cfdebd81907d14f0a4f35c3cfd2679a1bad
child 85553 a4abb0dd6b5643d8931928412ee437e3aec47567
push id805
push userakeybl@mozilla.com
push dateWed, 01 Feb 2012 18:17:35 +0000
treeherdermozilla-aurora@6fb3bf232436 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersUnfocused
bugs715735
milestone12.0a1
Bug 715735 - Detail view icon has no dimensions set, causes a relayout when it loads; r=Unfocused
toolkit/mozapps/extensions/content/extensions.xul
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.xul
+++ b/toolkit/mozapps/extensions/content/extensions.xul
@@ -523,17 +523,19 @@
                   <button id="detail-undo-btn" class="button-link"
                           label="&addon.undoAction.label;"
                           tooltipText="&addon.undoAction.tooltip;"
                           command="cmd_cancelOperation"/>
                   <spacer flex="5000"/> <!-- Necessary to allow the message to wrap -->
                 </hbox>
               </vbox>
               <hbox align="start">
-                <image id="detail-icon" class="icon"/>
+                <vbox id="detail-icon-container" align="end">
+                  <image id="detail-icon" class="icon"/>
+                </vbox>
                 <vbox flex="1">
                   <vbox id="detail-summary">
                     <hbox id="detail-name-container" class="name-container"
                           align="start">
                       <label id="detail-name" flex="1"/>
                       <label id="detail-version"/>
                       <label class="disabled-postfix" value="&addon.disabled.postfix;"/>
                       <label class="update-postfix" value="&addon.update.postfix;"/>
--- a/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
@@ -643,18 +643,22 @@
 }
 
 #detail-notifications .warning,
 #detail-notifications .pending,
 #detail-notifications .error {
   -moz-margin-start: 0;
 }
 
+#detail-icon-container {
+  width: 64px;
+  -moz-margin-end: 10px;
+}
+
 #detail-icon {
-  -moz-margin-end: 10px;
   max-width: 64px;
   max-height: 64px;
 }
 
 #detail-summary {
   margin-bottom: 2em;
 }
 
--- a/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
@@ -782,19 +782,23 @@
 }
 
 #detail-notifications .warning,
 #detail-notifications .pending,
 #detail-notifications .error {
   -moz-margin-start: 0;
 }
 
-#detail-icon {
+#detail-icon-container {
+  width: 64px;
   -moz-margin-end: 10px;
   margin-top: 6px;
+}
+
+#detail-icon {
   max-width: 64px;
   max-height: 64px;
 }
 
 #detail-summary {
   margin-bottom: 2em;
 }
 
--- a/toolkit/themes/winstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/winstripe/mozapps/extensions/extensions.css
@@ -760,19 +760,23 @@
 }
 
 #detail-notifications .warning,
 #detail-notifications .pending,
 #detail-notifications .error {
   -moz-margin-start: 0;
 }
 
-#detail-icon {
+#detail-icon-container {
+  width: 64px;
   -moz-margin-end: 10px;
   margin-top: 6px;
+}
+
+#detail-icon {
   max-width: 64px;
   max-height: 64px;
 }
 
 #detail-summary {
   margin-bottom: 2em;
 }