Bug 1571663 - fix height of attachment pane. r=Paenglab a=jorgk
authorAlessandro Castellani <alessandro@thunderbird.net>
Tue, 06 Aug 2019 14:19:26 -0700
changeset 35284 16a23290aead7ae74a967f509e1fc14bdf558b6a
parent 35283 dadc53787af48f91dfecd8f82973acc7e4908395
child 35285 8dd24940ccfa4d13dad9490e14a875634de79be7
push id2477
push usermozilla@jorgk.com
push dateTue, 13 Aug 2019 21:14:23 +0000
treeherdercomm-beta@436d8b620b6a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersPaenglab, jorgk
bugs1571663
Bug 1571663 - fix height of attachment pane. r=Paenglab a=jorgk
mail/base/content/attachmentList.css
mail/base/content/msgAttachmentView.inc.xul
--- a/mail/base/content/attachmentList.css
+++ b/mail/base/content/attachmentList.css
@@ -1,12 +1,16 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+.attachments-container {
+  display: block;
+}
+
 .attachmentList {
   -moz-appearance: listbox;
   -moz-user-focus: normal;
   margin: 0;
   padding: 3px;
 }
 
 .attachmentList[orient="horizontal"] {
--- a/mail/base/content/msgAttachmentView.inc.xul
+++ b/mail/base/content/msgAttachmentView.inc.xul
@@ -89,18 +89,19 @@
                                               label="&deleteAllAttachmentsCmd.label;"
                                               accesskey="&deleteAllAttachmentsCmd.accesskey;"/>
                                   </menupopup>
                                 </toolbarbutton>
                               </toolbaritem>
                             </hbox>
                           </vbox>
                         </hbox>
-                        <richlistbox is="attachment-list" id="attachmentList"
-                                     class="attachmentList"
-                                     orient="horizontal"
-                                     seltype="multiple"
-                                     flex="1"
-                                     context="attachmentListContext"
-                                     itemcontext="attachmentItemContext"
-                                     role="listbox"
-                                     ondragstart="nsDragAndDrop.startDrag(event, attachmentListDNDObserver);"
-                                     ondragover="nsDragAndDrop.dragOver(event, attachmentListDNDObserver);"/>
+                        <hbox class="attachments-container">
+                          <richlistbox is="attachment-list" id="attachmentList"
+                                       class="attachmentList"
+                                       orient="horizontal"
+                                       seltype="multiple"
+                                       context="attachmentListContext"
+                                       itemcontext="attachmentItemContext"
+                                       role="listbox"
+                                       ondragstart="nsDragAndDrop.startDrag(event, attachmentListDNDObserver);"
+                                       ondragover="nsDragAndDrop.dragOver(event, attachmentListDNDObserver);"/>
+                        </hbox>