Bug 1584720 - follow-up to bug 1534913 to let search-textbox have a context menu. r=jorgk
authorMagnus Melin <mkmelin+mozilla@iki.fi>
Sat, 28 Sep 2019 23:31:54 +0200
changeset 27783 6881607913e56d63be56ec69fa934103d0bfbf75
parent 27782 440d884740e7c23eb1ed96a3b711de4b174b13c5
child 27784 dd116f5f342a890a69212169f4008600b2529749
push id16485
push usermozilla@jorgk.com
push dateSat, 28 Sep 2019 21:33:35 +0000
treeherdercomm-central@6881607913e5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjorgk
bugs1584720, 1534913
Bug 1584720 - follow-up to bug 1534913 to let search-textbox have a context menu. r=jorgk
mail/base/content/FilterListDialog.xul
mail/components/downloads/content/aboutDownloads.xul
mail/components/preferences/passwordManager.xul
--- a/mail/base/content/FilterListDialog.xul
+++ b/mail/base/content/FilterListDialog.xul
@@ -17,16 +17,18 @@
   windowtype="mailnews:filterlist"
   style="width: 45em; height: 31em;"
   onload="onLoad();"
   onunload="onFilterUnload();"
   onclose="return onFilterClose();"
   persist="width height screenX screenY"
   title="&window.title;">
 
+  <script src="chrome://global/content/globalOverlay.js"/>
+  <script src="chrome://global/content/editMenuOverlay.js"/>
   <script src="chrome://messenger/content/button-menu-button.js"/>
   <script src="chrome://messenger/content/searchWidgets.js"/>
   <script src="chrome://messenger/content/FilterListDialog.js"/>
   <script src="chrome://messenger/content/customElements.js"/>
   <stringbundleset id="stringbundleset">
     <stringbundle id="bundle_filter" src="chrome://messenger/locale/filter.properties"/>
     <stringbundle id="bundle_brand" src="chrome://branding/locale/brand.properties"/>
   </stringbundleset>
@@ -49,17 +51,17 @@
                  showFileHereLabel="true"
                  showAccountsFileHere="true"
                  oncommand="setFilterFolder(event.target._folder);"/>
     </menulist>
     <search-textbox id="searchBox"
                     class="searchBox"
                     flex="1"
                     oncommand="rebuildFilterList();"
-                    emptytext="&searchBox.emptyText;"
+                    placeholder="&searchBox.emptyText;"
                     isempty="true"/>
   </hbox>
   <separator class="thin"/>
   <hbox id="filterListGrid" flex="1">
     <vbox flex="1">
       <hbox>
         <label id="filterListLabel" control="filterList" flex="1">
           &filterHeader.label;
--- a/mail/components/downloads/content/aboutDownloads.xul
+++ b/mail/components/downloads/content/aboutDownloads.xul
@@ -15,16 +15,17 @@
 <window id="aboutDownloads"
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" xmlns:html="http://www.w3.org/1999/xhtml"
         xmlns:xhtml="http://www.w3.org/1999/xhtml"
         title="&aboutDownloads.title;"
         onload="DownloadsView.init();">
   <xhtml:link rel="shortcut icon"
               href="chrome://messenger/skin/downloads/download.svg"/>
   <script src="chrome://global/content/globalOverlay.js"/>
+  <script src="chrome://global/content/editMenuOverlay.js"/>
   <script src="chrome://messenger/content/downloads/aboutDownloads.js"/>
 
   <commandset id="msgDownloadCommands"
               commandupdater="true"
               events="focus,select,contextmenu">
     <command id="msgDownloadsCmd_open"
              oncommand="goDoCommand('msgDownloadsCmd_open')"/>
     <command id="msgDownloadsCmd_show"
--- a/mail/components/preferences/passwordManager.xul
+++ b/mail/components/preferences/passwordManager.xul
@@ -14,16 +14,18 @@
         onunload="Shutdown();"
         data-l10n-id="saved-logins"
         style="width: 45em;"
         persist="width height screenX screenY">
 
   <linkset>
     <html:link rel="localization" href="messenger/preferences/passwordManager.ftl"/>
   </linkset>
+  <script src="chrome://global/content/globalOverlay.js"/>
+  <script src="chrome://global/content/editMenuOverlay.js"/>
   <script src="chrome://messenger/content/preferences/passwordManager.js"/>
 
   <keyset>
     <key keycode="VK_ESCAPE" oncommand="escapeKeyHandler();"/>
     <key data-l10n-id="window-close" modifiers="accel" oncommand="escapeKeyHandler();"/>
     <key data-l10n-id="focus-search-shortcut" modifiers="accel" oncommand="FocusFilterBox();"/>
     <key data-l10n-id="focus-search-altshortcut" modifiers="accel" oncommand="FocusFilterBox();"/>
   </keyset>