Bug 657343 - Vertical misalignment of message headers which don't have an associated link r=mnyromyr a=iann CLOSED TREE
authorneil@httl.net
Thu, 09 Jun 2011 21:21:03 +0100
changeset 7807 0c7f86ea92f73ec8ce78149f402b4f20889c8814
parent 7806 e82715ab3680667e95edc8790c355393356ce499
child 7808 48b34c3d35b6d854e67a7836c85422350bcc7775
push id13
push useriann_cvs@blueyonder.co.uk
push dateThu, 09 Jun 2011 20:19:54 +0000
treeherdercomm-beta@0c7f86ea92f7 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmnyromyr, iann
bugs657343
Bug 657343 - Vertical misalignment of message headers which don't have an associated link r=mnyromyr a=iann CLOSED TREE
suite/mailnews/mailWidgets.xml
--- a/suite/mailnews/mailWidgets.xml
+++ b/suite/mailnews/mailWidgets.xml
@@ -161,49 +161,55 @@
        Use headerValue to set the header value. -->
   <binding id="mail-toggle-headerfield">
     <content>
       <xul:hbox class="headerNameBox" align="start">
         <xul:image class="expandHeaderViewButton" xbl:inherits="onclick=ontwistyclick"/>
         <xul:spacer flex="1"/>
         <xul:label class="headerName" xbl:inherits="value=label" control="headerValue"/>
       </xul:hbox>
-      <xul:textbox class="headerValue plain" anonid="headerValue" flex="1" readonly="true"/>
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:textbox class="headerValue plain" anonid="headerValue" flex="1" readonly="true"/>
+      </xul:hbox>
     </content>
 
     <implementation>
       <property name="headerValue" onset="return document.getAnonymousElementByAttribute(this, 'anonid', 'headerValue').value = val;"/>
     </implementation>
   </binding>
 
   <!-- mail-headerfield: presents standard text header name & value pairs. Don't use this for email addresses.
        use label to set the header name.
        use headerValue to set the header value. -->
   <binding id="mail-headerfield">
     <content>
       <xul:hbox class="headerNameBox" align="start">
         <xul:label class="headerName" xbl:inherits="value=label" control="headerValue" flex="1"/>
       </xul:hbox>
-      <xul:textbox class="headerValue plain" anonid="headerValue" flex="1" readonly="true"/>
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:textbox class="headerValue plain" anonid="headerValue" flex="1" readonly="true"/>
+      </xul:hbox>
     </content>
 
     <implementation>
       <property name="headerValue" onset="return document.getAnonymousElementByAttribute(this, 'anonid', 'headerValue').value = val;"/>
     </implementation>
   </binding>
 
   <binding id="mail-urlfield" extends="chrome://messenger/content/mailWidgets.xml#mail-headerfield">
     <content>
       <xul:hbox class="headerNameBox" align="start">
         <xul:label class="headerName" xbl:inherits="value=label" flex="1"/>
       </xul:hbox>
-      <xul:label onclick="if (event.button != 2) openUILink(event.target.value, event);"
-                 ondragstart="this.parentNode.setDataTransfer(event);"
-                 class="headerValue plain text-link headerValueUrl"
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:label onclick="if (event.button != 2) openUILink(event.target.value, event);"
+                   ondragstart="this.parentNode.setDataTransfer(event);"
+                   class="headerValue plain text-link headerValueUrl"
                  anonid="headerValue" flex="1" readonly="true" context="copyUrlPopup"/>
+      </xul:hbox>
     </content>
     
     <implementation>
       <method name="setDataTransfer">
         <parameter name="aEvent"/>
         <body><![CDATA[
           var dt = aEvent.dataTransfer;
           var val = aEvent.target.value;
@@ -215,17 +221,19 @@
     </implementation>
   </binding>
 
   <binding id="mail-emailheaderfield">
     <content>
       <xul:hbox class="headerNameBox" align="start">
         <xul:label class="headerName" xbl:inherits="value=label" flex="1"/>
       </xul:hbox>
-      <xul:mail-emailaddress class="headerValue" anonid="emailAddressNode"/>
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:mail-emailaddress class="headerValue" anonid="emailAddressNode"/>
+      </xul:hbox>
     </content>
 
     <implementation>
       <property name="emailAddressNode" onget="return document.getAnonymousElementByAttribute(this, 'anonid', 'emailAddressNode');"
         readonly="true"/>
     </implementation>
   </binding>
 
@@ -233,17 +241,17 @@
   <binding id="mail-multi-emailHeaderField">
     <content>
       <xul:hbox class="headerNameBox" align="start" pack="end">
         <xul:image class="addresstwisty" anonid="toggleIcon"
                    collapsed="true" onclick="toggleWrap();"/>
         <xul:label class="headerName" xbl:inherits="value=label"/>
       </xul:hbox>
 
-      <xul:hbox class="headerValueBox" anonid="longEmailAddresses" flex="1"
+      <xul:hbox class="headerValueBox" anonid="longEmailAddresses" flex="1" align="start"
                 onoverflow="if (event.detail != 1) this.parentNode.toggleIcon.collapsed = false;"
                 onunderflow="if (event.detail != 1) this.parentNode.toggleIcon.collapsed = true;">
         <xul:description class="headerValue" anonid="emailAddresses" flex="1"/>
       </xul:hbox>
     </content>
 
     <implementation>
       <constructor>
@@ -516,18 +524,18 @@
 
   <binding id="mail-messageids-headerfield">
     <content>
       <xul:hbox class="headerNameBox" align="start" pack="end">
         <xul:image class="addresstwisty" anonid="toggleIcon"
                    onclick="toggleWrap();"/>
         <xul:label class="headerName" xbl:inherits="value=label"/>
       </xul:hbox>
-      <xul:hbox class="headerValueBox" flex="1">
-      <xul:label class="headerValue" anonid="headerValue" flex="1"/>
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:label class="headerValue" anonid="headerValue" flex="1"/>
       </xul:hbox>
     </content>
 
     <implementation>
       <constructor>
         <![CDATA[
           this.mMessageIds = [];
           this.showFullMessageIds = false;
@@ -684,17 +692,19 @@
   </binding>
 
   <!-- Header field for showing the tags associated with a message -->
   <binding id="mail-headerfield-tags">
     <content>
       <xul:hbox class="headerNameBox" align="start">
         <xul:label class="headerName" xbl:inherits="value=label" flex="1"/>
       </xul:hbox>
-      <xul:label class="headerValue plain" anonid="headerValue" flex="1"/>
+      <xul:hbox class="headerValueBox" flex="1" align="start">
+        <xul:label class="headerValue plain" anonid="headerValue" flex="1"/>
+      </xul:hbox>
     </content>
 
     <implementation>
       <property name="headerValue" onset="return this.buildTags(val);"/>
       <method name="buildTags">
         <parameter name="aTags"/>
         <body>
           <![CDATA[