Bug 1559127 - Disable the appmenu's "Go > Recently Closed Tabs" when needed. r=darktrojan a=jorgk BETA_68_CONTINUATION
authorPaul Morris <paul@paulwmorris.com>
Wed, 03 Jul 2019 16:59:28 -0400
branchBETA_68_CONTINUATION
changeset 36071 658bff9d5cc55ec3bc9c0c4280127d78773ee064
parent 36070 feabcfb77b987338e3a28a8b7d6440ff9e0e2e23
child 36072 bd356636aab529d2137c9a0a2f1c06244962d780
push id392
push userclokep@gmail.com
push dateMon, 02 Sep 2019 20:17:19 +0000
reviewersdarktrojan, jorgk
bugs1559127
Bug 1559127 - Disable the appmenu's "Go > Recently Closed Tabs" when needed. r=darktrojan a=jorgk
mail/components/customizableui/content/panelUI.inc.xul
mail/components/customizableui/moz.build
--- a/mail/components/customizableui/content/panelUI.inc.xul
+++ b/mail/components/customizableui/content/panelUI.inc.xul
@@ -1306,16 +1306,17 @@
                        class="subviewbutton subviewbutton-nav"
                        label="&folderMenu.label;"
                        closemenu="none"
                        oncommand="PanelUI.showSubView('appMenu-goFolderView', this)"/>
         <toolbarseparator id="appmenu_goFolderSeparator"/>
         <toolbarbutton id="appmenu_goRecentlyClosedTabs"
                        class="subviewbutton subviewbutton-nav"
                        label="&goRecentlyClosedTabs.label;"
+                       observes="cmd_undoCloseTab"
                        closemenu="none"
                        oncommand="PanelUI.showSubView('appMenu-goRecentlyClosedTabsView', this)"/>
         <toolbarseparator id="appmenu_goRecentlyClosedTabsSeparator"/>
         <toolbarbutton id="appmenu_goStartPage"
                        class="subviewbutton subviewbutton-iconic"
                        label="&startPageCmd.label;"
                        key="key_goStartPage"
                        command="cmd_goStartPage"/>
@@ -1383,21 +1384,16 @@
                showRecent="true"
                showFavorites="true"
                command="cmd_goFolder"
                recentLabel="&contextMoveCopyMsgRecentMenu.label;"
                favoritesLabel="&contextMoveCopyMsgFavoritesMenu.label;"/>
 
     <!-- Go / Recently Closed Tabs -->
     <!-- Dynamically populated when shown. -->
-    <!-- TODO appmenu - what about this 'observes' bit?
-        <menu id="appmenu_goRecentlyClosedTabs"
-                  label="&goRecentlyClosedTabs.label;"
-                  observes="cmd_undoCloseTab">
-    -->
     <panelview id="appMenu-goRecentlyClosedTabsView"
                title="&goRecentlyClosedTabs.label;"
                class="PanelUI-subView">
       <vbox class="panel-subview-body"/>
     </panelview>
 
 
     <!-- Message -->
--- a/mail/components/customizableui/moz.build
+++ b/mail/components/customizableui/moz.build
@@ -3,24 +3,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/.
 
 DIRS += [
     'content',
 ]
 
-# TODO appmenu - Not currently used for Thunderbird.
-# BROWSER_CHROME_MANIFESTS += ['test/browser.ini']
-
-# TODO appmenu - Not currently used for Thunderbird.
-# TESTING_JS_MODULES += [
-#     'test/CustomizableUITestUtils.jsm',
-# ]
-
 EXTRA_JS_MODULES += [
     'CustomizableUI.jsm',
     'CustomizableWidgets.jsm',
     'CustomizeMode.jsm',
     'DragPositionManager.jsm',
     'PanelMultiView.jsm',
     'SearchWidgetTracker.jsm',
 ]