From adcc3950fb98a30e4bc0355a6a6b03274d075966 Mon Sep 17 00:00:00 2001
From: "shliang%netscape.com" <shliang%netscape.com>
Date: Fri, 14 Mar 2003 06:27:48 +0000
Subject: [PATCH] 147653 - page setup commands doesn't work in mail and
 composer. r/sr=sspitzer,jag

---
 editor/ui/composer/content/ComposerCommands.js                | 2 +-
 mailnews/compose/resources/content/MsgComposeCommands.js      | 4 ++--
 mailnews/compose/resources/content/messengercompose.xul       | 2 +-
 xpfe/browser/resources/content/navigator.js                   | 2 +-
 xpfe/browser/resources/content/navigatorOverlay.xul           | 4 ++--
 xpfe/browser/resources/content/viewSourceOverlay.xul          | 2 +-
 .../resources/content/mac/platformCommunicatorOverlay.xul     | 4 ++--
 xpfe/communicator/resources/content/utilityOverlay.xul        | 3 ---
 8 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/editor/ui/composer/content/ComposerCommands.js b/editor/ui/composer/content/ComposerCommands.js
index 45bccc92511b..5182e98a53f9 100644
--- a/editor/ui/composer/content/ComposerCommands.js
+++ b/editor/ui/composer/content/ComposerCommands.js
@@ -2297,7 +2297,7 @@ var nsPrintSetupCommand =
   {
     // In editor.js
     FinishHTMLSource();
-    goPageSetup();
+    NSPrintSetup();
   }
 };
 
diff --git a/mailnews/compose/resources/content/MsgComposeCommands.js b/mailnews/compose/resources/content/MsgComposeCommands.js
index 146d28e8b1be..c84e50c0e834 100644
--- a/mailnews/compose/resources/content/MsgComposeCommands.js
+++ b/mailnews/compose/resources/content/MsgComposeCommands.js
@@ -477,8 +477,8 @@ var defaultController =
       case "cmd_sendNow"            : if (defaultController.isCommandEnabled(command)) SendMessage();          break;
       case "cmd_sendWithCheck"   : if (defaultController.isCommandEnabled(command)) SendMessageWithCheck();          break;
       case "cmd_sendLater"          : if (defaultController.isCommandEnabled(command)) SendMessageLater();     break;
-      case "cmd_printSetup"         : NSPrintSetup();                                                           break;
-      case "cmd_print"              : DoCommandPrint();                                                        break;
+      case "cmd_printSetup"         : NSPrintSetup(); break;
+      case "cmd_print"              : DoCommandPrint(); break;
 
       //Edit Menu
       case "cmd_delete"             : if (MessageHasSelectedAttachments()) RemoveSelectedAttachment();         break;
diff --git a/mailnews/compose/resources/content/messengercompose.xul b/mailnews/compose/resources/content/messengercompose.xul
index 5dc36407768c..764cfb65b281 100644
--- a/mailnews/compose/resources/content/messengercompose.xul
+++ b/mailnews/compose/resources/content/messengercompose.xul
@@ -229,7 +229,7 @@
           <menuitem label="&sendNowCmd.label;" accesskey="&sendNowCmd.accesskey;" key="key_send" command="cmd_sendNow" id="menu-item-send-now"/>
           <menuitem label="&sendLaterCmd.label;" accesskey="&sendLaterCmd.accesskey;" key="key_sendLater" command="cmd_sendLater"/>
           <menuseparator/>
-          <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="Browser:PrintSetup"/>
+          <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="cmd_printSetup"/>
           <menuitem id="printMenuItem" label="&printCmd.label;" accesskey="&printCmd.accesskey;" key="key_print" command="cmd_print"/>
         </menupopup>
       </menu>
diff --git a/xpfe/browser/resources/content/navigator.js b/xpfe/browser/resources/content/navigator.js
index 6bc1ba36fb5f..769cd84192cf 100644
--- a/xpfe/browser/resources/content/navigator.js
+++ b/xpfe/browser/resources/content/navigator.js
@@ -1486,7 +1486,7 @@ function hiddenWindowStartup()
   window.focus();
 
   // Disable menus which are not appropriate
-  var disabledItems = ['cmd_close', 'Browser:SendPage', 'Browser:EditPage', 'Browser:PrintSetup', /*'Browser:PrintPreview',*/
+  var disabledItems = ['cmd_close', 'Browser:SendPage', 'Browser:EditPage', 'cmd_printSetup', /*'Browser:PrintPreview',*/
                        'Browser:Print', 'canGoBack', 'canGoForward', 'Browser:Home', 'Browser:AddBookmark', 'cmd_undo',
                        'cmd_redo', 'cmd_cut', 'cmd_copy','cmd_paste', 'cmd_delete', 'cmd_selectAll', 'menu_textZoom'];
   for (var id in disabledItems) {
diff --git a/xpfe/browser/resources/content/navigatorOverlay.xul b/xpfe/browser/resources/content/navigatorOverlay.xul
index 7d22a79c2a10..a58c31a74740 100644
--- a/xpfe/browser/resources/content/navigatorOverlay.xul
+++ b/xpfe/browser/resources/content/navigatorOverlay.xul
@@ -121,7 +121,7 @@
     <command id="Browser:SavePage" oncommand="saveDocument(window._content.document);"/>
     <command id="Browser:EditPage" oncommand="editPageOrFrame();" observes="isImage"/>
     <command id="Browser:Open" oncommand="BrowserOpenWindow();"/>
-    <command id="Browser:PrintSetup" oncommand="NSPrintSetup();"/>
+    <command id="cmd_printSetup" oncommand="NSPrintSetup();"/>
     <command id="Browser:Print" oncommand="NSPrint();"/>
     <command id="Browser:PrintPreview" oncommand="BrowserPrintPreview();"/>
     <command id="cmd_quit"/>
@@ -215,7 +215,7 @@
         <menuseparator id="saveMenuBlockEnd"/>
         <menuitem label="&editPageCmd.label;" accesskey="&editPageCmd.accesskey;" key="key_editPage" command="Browser:EditPage" />
         <menuseparator/>
-        <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="Browser:PrintSetup"/>
+        <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="cmd_printSetup"/>
         <menuitem id="printPreviewMenuItem" label="&printPreviewCmd.label;" accesskey="&printPreviewCmd.accesskey;" command="Browser:PrintPreview"/>
         <menuitem id="printMenuItem" label="&printCmd.label;" accesskey="&printCmd.accesskey;" key="printKb" command="Browser:Print"/>
         <menuseparator/>
diff --git a/xpfe/browser/resources/content/viewSourceOverlay.xul b/xpfe/browser/resources/content/viewSourceOverlay.xul
index 80f4206dde46..32e5ae736c51 100644
--- a/xpfe/browser/resources/content/viewSourceOverlay.xul
+++ b/xpfe/browser/resources/content/viewSourceOverlay.xul
@@ -139,7 +139,7 @@
                     label="&editPageCmd.label;" accesskey="&editPageCmd.accesskey;"/>
           <menuseparator/>
           <menuitem label="&printCmd.label;" accesskey="&printCmd.accesskey;" key="printKb" command="Browser:Print"/>     
-          <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="Browser:PrintSetup"/>
+          <menuitem id="printSetupMenuItem" label="&printSetupCmd.label;" accesskey="&printSetupCmd.accesskey;" command="cmd_printSetup"/>
         </menupopup>
       </menu>
 
diff --git a/xpfe/communicator/resources/content/mac/platformCommunicatorOverlay.xul b/xpfe/communicator/resources/content/mac/platformCommunicatorOverlay.xul
index f6db26e22e52..693cc39ffb83 100644
--- a/xpfe/communicator/resources/content/mac/platformCommunicatorOverlay.xul
+++ b/xpfe/communicator/resources/content/mac/platformCommunicatorOverlay.xul
@@ -25,7 +25,7 @@
          insertbefore="printMenuItem"
          label="&printSetupCmd.label;"
          accesskey="&printSetupCmd.accesskey;"
-         command="Browser:PrintSetup"/>
+         command="cmd_printSetup"/>
     <!-- quit -->
     <menuseparator id="menu_FileQuitSeparator"/>
     <menuitem label="&quitApplicationCmd.label;" id="menu_FileQuitItem"
@@ -51,7 +51,7 @@
          insertafter="printMenuItemToolbar"
          label="&printSetupCmd.label;"
          accesskey="&printSetupCmd.accesskey;"
-         command="Browser:PrintSetup"/>
+         command="cmd_printSetup"/>
   </menupopup>
 
   <!-- Find As You Type Keys -->
diff --git a/xpfe/communicator/resources/content/utilityOverlay.xul b/xpfe/communicator/resources/content/utilityOverlay.xul
index 4f9649d9c7ba..90ec96268162 100644
--- a/xpfe/communicator/resources/content/utilityOverlay.xul
+++ b/xpfe/communicator/resources/content/utilityOverlay.xul
@@ -55,9 +55,6 @@
   <command id="cmd_newEditorDraft"
            disabled="true"
            oncommand="NewEditorFromDraft();"/>
-  <command id="cmd_pageSetup"
-           disabled="true"
-           oncommand="goPageSetup();"/>
   -->
 
   <menuitem id="menu_newEditor"