Bug 1401754 - print in the hamburger panel should open preview on Windows/Linux but not suggest its shortcut is ctrl-p, r?dao draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Wed, 20 Sep 2017 22:54:38 +0100
changeset 667940 194d3cae1b9c79cecde3d26f7f996135bb871567
parent 667637 0c49ebf4195ce51072b24aacf01eaaffdcff9be6
child 732544 4fdf1228e52da6d2c6bd89f568f5767e7b43733b
push id80881
push userbmo:gijskruitbosch+bugs@gmail.com
push dateWed, 20 Sep 2017 21:56:08 +0000
reviewersdao
bugs1401754
milestone57.0a1
Bug 1401754 - print in the hamburger panel should open preview on Windows/Linux but not suggest its shortcut is ctrl-p, r?dao MozReview-Commit-ID: K4yQB6ejGRv
browser/components/customizableui/content/panelUI.inc.xul
--- a/browser/components/customizableui/content/panelUI.inc.xul
+++ b/browser/components/customizableui/content/panelUI.inc.xul
@@ -640,18 +640,22 @@
                        class="subviewbutton"
                        label="&savePageCmd.label;"
                        key="key_savePage"
                        command="Browser:SavePage"
                        />
         <toolbarbutton id="appMenu-print-button"
                        class="subviewbutton subviewbutton-iconic"
                        label="&printCmd.label;"
+#ifdef XP_MACOSX
                        key="printKb"
                        command="cmd_print"
+#else
+                       command="cmd_printPreview"
+#endif
                        />
         <toolbarseparator/>
         <toolbarbutton id="appMenu-find-button"
                        class="subviewbutton subviewbutton-iconic"
                        label="&findOnCmd.label;"
                        key="key_find"
                        command="cmd_find"/>
         <toolbarbutton id="appMenu-more-button"