Bug 1374477 - Remove page actions from browser.xul. r?mikedeboer
MozReview-Commit-ID: 5nlXfKOx3Aq
--- a/browser/base/content/browser-sets.inc
+++ b/browser/base/content/browser-sets.inc
@@ -23,21 +23,16 @@
<command id="cmd_newNavigatorTab" oncommand="BrowserOpenTab(event);"/>
<command id="cmd_newNavigatorTabNoEvent" oncommand="BrowserOpenTab();"/>
<command id="Browser:OpenFile" oncommand="BrowserOpenFileWindow();"/>
<command id="Browser:SavePage" oncommand="saveBrowser(gBrowser.selectedBrowser);"/>
<command id="Browser:SendLink"
oncommand="MailIntegration.sendLinkForBrowser(gBrowser.selectedBrowser);"/>
- <command id="PageAction:copyURL"
- oncommand="gPageActionButton.copyURL();"/>
- <command id="PageAction:emailLink"
- oncommand="gPageActionButton.emailLink();"/>
-
<command id="cmd_pageSetup" oncommand="PrintUtils.showPageSetup();"/>
<command id="cmd_print" oncommand="PrintUtils.printWindow(window.gBrowser.selectedBrowser.outerWindowID, window.gBrowser.selectedBrowser);"/>
<command id="cmd_printPreview" oncommand="PrintUtils.printPreview(PrintPreviewListener);"/>
<command id="cmd_close" oncommand="BrowserCloseTabOrWindow()"/>
<command id="cmd_closeWindow" oncommand="BrowserTryToCloseWindow()"/>
<command id="cmd_toggleMute" oncommand="gBrowser.selectedTab.toggleMuteAudio()"/>
<command id="cmd_CustomizeToolbars" oncommand="BrowserCustomizeToolbar()"/>
<command id="cmd_quitApplication" oncommand="goQuitApplication()"/>
--- a/browser/base/content/browser.xul
+++ b/browser/base/content/browser.xul
@@ -442,72 +442,55 @@
<button class="ctrlTab-preview" flex="1"/>
<button class="ctrlTab-preview" flex="1"/>
</hbox>
<hbox pack="center">
<button id="ctrlTab-showAll" class="ctrlTab-preview" noicon="true"/>
</hbox>
</panel>
- <panel id="page-action-panel"
+#ifdef MOZ_PHOTON_THEME
+ <panel id="pageActionPanel"
class="cui-widget-panel"
role="group"
type="arrow"
hidden="true"
flip="slide"
position="bottomcenter topright"
- noautofocus="true">
- <photonpanelmultiview id="page-action-multiView"
- mainViewId="page-action-mainView">
- <panelview id="page-action-mainView"
+ context="pageActionPanelContextMenu"
+ oncontextmenu="BrowserPageActions.onContextMenu(event);"
+ noautofocus="true"
+ copyURL-title="©URLCmd.label;"
+ emailLink-title="&emailPageCmd.label;"
+ sendToDevice-title="&sendToDevice.label2;"
+ sendToDevice-subviewTitle="&sendToDevice.viewTitle;"
+ sendToDevice-fxaTitle="&syncBrand.fxAccount.label;"
+ sendToDevice-fxaShortcut="&sendToDevice.fxaRequired.label;"
+ sendToDevice-noDevicesTitle="&sendToDevice.noDevices.label;"
+ sendToDevice-notReadyTitle="&sendToDevice.syncNotReady.label;">
+ <photonpanelmultiview id="pageActionPanelMultiView"
+ mainViewId="pageActionPanelMainView"
+ viewCacheId="appMenu-viewCache">
+ <panelview id="pageActionPanelMainView"
class="PanelUI-subView">
- <vbox class="panel-subview-body">
- <toolbarbutton id="page-action-bookmark-button"
- class="subviewbutton subviewbutton-iconic"
- observes="bookmarkThisPageBroadcaster"
- command="Browser:AddBookmarkAs"
- onclick="gPageActionButton.panel.hidePopup();"/>
- <toolbarseparator/>
- <toolbarbutton id="page-action-copy-url-button"
- class="subviewbutton subviewbutton-iconic"
- label="©URLCmd.label;"
- command="PageAction:copyURL"/>
- <toolbarbutton id="page-action-email-link-button"
- class="subviewbutton subviewbutton-iconic"
- label="&emailPageCmd.label;"
- command="PageAction:emailLink"/>
- <toolbarbutton id="page-action-send-to-device-button"
- class="subviewbutton subviewbutton-iconic subviewbutton-nav"
- label="&sendToDevice.label2;"
- closemenu="none"
- oncommand="gPageActionButton.showSendToDeviceView(this);"/>
- </vbox>
- </panelview>
- <panelview id="page-action-sendToDeviceView"
- class="PanelUI-subView"
- title="&sendToDevice.viewTitle;">
- <vbox id="page-action-sendToDeviceView-body" class="panel-subview-body">
- <toolbarbutton id="page-action-sendToDevice-fxa-button"
- class="subviewbutton subviewbutton-iconic"
- label="&syncBrand.fxAccount.label;"
- shortcut="&sendToDevice.fxaRequired.label;"
- oncommand="gPageActionButton.fxaButtonClicked();"/>
- <toolbarbutton id="page-action-no-devices-button"
- class="subviewbutton"
- label="&sendToDevice.noDevices.label;"
- disabled="true"/>
- <toolbarbutton id="page-action-sync-not-ready-button"
- class="subviewbutton"
- label="&sendToDevice.syncNotReady.label;"
- disabled="true"/>
- </vbox>
+ <vbox class="panel-subview-body"/>
</panelview>
</photonpanelmultiview>
</panel>
+ <menupopup id="pageActionPanelContextMenu"
+ onpopupshowing="BrowserPageActions.onContextMenuShowing(event, this);">
+ <menuitem id="pageActionPanelContextMenu-toggleUrlbar"
+ add-label="&pageAction.addToUrlbar.label;"
+ remove-label="&pageAction.removeFromUrlbar.label;"
+ label="&pageAction.addToUrlbar.label;"
+ oncommand="BrowserPageActions.toggleShownInUrlbarForContextAction();"/>
+ </menupopup>
+#endif
+
<!-- Bookmarks and history tooltip -->
<tooltip id="bhTooltip"/>
<tooltip id="tabbrowser-tab-tooltip" onpopupshowing="gBrowser.createTooltip(event);"/>
<tooltip id="back-button-tooltip">
<label class="tooltip-label" value="&backButton.tooltip;"/>
#ifdef XP_MACOSX
@@ -913,24 +896,16 @@
class="urlbar-icon"
hidden="true"
tooltiptext="&pageReportIcon.tooltip;"
onmousedown="gPopupBlockerObserver.onReportButtonMousedown(event);"/>
<image id="reader-mode-button"
class="urlbar-icon"
hidden="true"
onclick="ReaderParent.buttonClick(event);"/>
-#ifdef MOZ_PHOTON_THEME
- <image id="star-button"
- class="urlbar-icon"
- onclick="BookmarkingUI.onStarCommand(event);">
- <observes element="bookmarkThisPageBroadcaster" attribute="starred"/>
- <observes element="bookmarkThisPageBroadcaster" attribute="tooltiptext"/>
- </image>
-#endif
<toolbarbutton id="urlbar-zoom-button"
onclick="FullZoom.reset();"
tooltip="dynamic-shortcut-tooltip"
hidden="true"/>
</hbox>
<hbox id="userContext-icons" hidden="true">
<label id="userContext-label"/>
<image id="userContext-indicator"/>
@@ -945,20 +920,22 @@
command="Browser:ReloadOrDuplicate"
onclick="checkForMiddleClick(this, event);"
tooltip="dynamic-shortcut-tooltip"/>
<toolbarbutton id="stop-button"
class="chromeclass-toolbar-additional"
command="Browser:Stop"
tooltip="dynamic-shortcut-tooltip"/>
#else
- <toolbarbutton id="urlbar-page-action-button"
- class="chromeclass-toolbar-additional"
- tooltiptext="&pageActionButton.tooltip;"
- onclick="gPageActionButton.onEvent(event);"/>
+ <hbox id="pageActionUrlbarIcons">
+ <image id="pageActionButton"
+ class="urlbar-icon"
+ tooltiptext="&pageActionButton.tooltip;"
+ onclick="BrowserPageActions.mainButtonClicked(event);"/>
+ </hbox>
#endif
</textbox>
</hbox>
</toolbaritem>
<toolbaritem id="search-container" title="&searchItem.title;"
align="center" class="chromeclass-toolbar-additional panel-wide-item"
cui-areatype="toolbar"
--- a/browser/locales/en-US/chrome/browser/browser.dtd
+++ b/browser/locales/en-US/chrome/browser/browser.dtd
@@ -980,14 +980,16 @@ you can use these alternative items. Oth
<!ENTITY updateRestart.header.message2 "Restart to update &brandShorterName;.">
<!ENTITY updateRestart.acceptButton.label "Restart and Restore">
<!ENTITY updateRestart.acceptButton.accesskey "R">
<!ENTITY updateRestart.cancelButton.label "Not Now">
<!ENTITY updateRestart.cancelButton.accesskey "N">
<!ENTITY updateRestart.panelUI.label2 "Restart to update &brandShorterName;">
<!ENTITY pageActionButton.tooltip "Page actions">
+<!ENTITY pageAction.addToUrlbar.label "Add to Address Bar">
+<!ENTITY pageAction.removeFromUrlbar.label "Remove from Address Bar">
<!ENTITY sendToDevice.label2 "Send to Device">
<!ENTITY sendToDevice.viewTitle "Send to Device">
<!ENTITY sendToDevice.fxaRequired.label "Required">
<!ENTITY sendToDevice.noDevices.label "No Devices Available">
<!ENTITY sendToDevice.syncNotReady.label "Syncing Devices…">