Bug 1590666 - Give the searchTerm list a minheight in filter editor. r=aleca a=jorgk
authorRichard Marti <richard.marti@gmail.com>
Mon, 28 Oct 2019 22:41:19 +0100
changeset 37229 14f12d4f3c1d0a60e9f728617934b91b718b845a
parent 37228 3aae0034e44c0fbc7f847b9cbdd5474b581c1d20
child 37230 892a6344acd28441c15d183feabe7776b58bd5db
push id395
push userclokep@gmail.com
push dateMon, 02 Dec 2019 19:38:57 +0000
reviewersaleca, jorgk
bugs1590666
Bug 1590666 - Give the searchTerm list a minheight in filter editor. r=aleca a=jorgk
mailnews/base/search/content/FilterEditor.xul
mailnews/base/search/content/searchTerm.inc.xul
--- a/mailnews/base/search/content/FilterEditor.xul
+++ b/mailnews/base/search/content/FilterEditor.xul
@@ -55,19 +55,17 @@
       <html:input id="filterName"
                   type="text"
                   class="input-inline"
                   aria-labelledby="filterNameLabel"/>
       <spacer class="spacer-inline"/>
     </hbox>
   </vbox>
 
-  <separator class="thin"/>
-
-  <vbox flex="1">
+  <vbox>
     <groupbox>
       <label class="header">&contextDesc.label;</label>
       <vbox>
         <hbox flex="1" align="center">
           <checkbox id="runManual"
                     label="&contextManual.label;"
                     accesskey="&contextManual.accesskey;"
                     command="cmd_updateFilterType"/>
@@ -102,20 +100,20 @@
         <hbox flex="1" align="center">
           <checkbox id="runPeriodic"
                     accesskey="&contextPeriodic.accesskey;"
                     command="cmd_updateFilterType"/>
           <label id="periodLength"/>
         </hbox>
       </vbox>
     </groupbox>
+  </vbox>
 
     <vbox id="searchTermListBox" flex="1">
 #include searchTerm.inc.xul
-  </vbox>
 
   <splitter id="gray_horizontal_splitter" persist="state" orient="vertical"/>
 
   <vbox flex="1">
     <label value="&filterActionDesc.label;"
            accesskey="&filterActionDesc.accesskey;"
            control="filterActionList"/>
     <richlistbox id="filterActionList"
--- a/mailnews/base/search/content/searchTerm.inc.xul
+++ b/mailnews/base/search/content/searchTerm.inc.xul
@@ -7,19 +7,19 @@
             <radio value="and" label="&matchAll.label;"
                    accesskey="&matchAll.accesskey;"/>
             <radio value="or" label="&matchAny.label;"
                    accesskey="&matchAny.accesskey;"/>
             <radio value="matchAll" id="matchAllItem" label="&matchAllMsgs.label;"
                    accesskey="&matchAllMsgs.accesskey;"/>
           </radiogroup>
 
-          <hbox flex="1">
+          <hbox flex="1" minheight="100">
             <hbox id="searchterms"/>
-            <richlistbox flex="1" id="searchTermList" minheight="35%">
+            <richlistbox flex="1" id="searchTermList">
               <treecols hidden="true">
                 <treecol flex="&searchTermListAttributesFlexValue;"/>
                 <treecol flex="&searchTermListOperatorsFlexValue;"/>
                 <treecol flex="&searchTermListValueFlexValue;"/>
                 <treecol class="filler"/>
               </treecols>
             </richlistbox>