From 3379b80f07e42f9c0bd8c57f3d50a3b51e3bb8b6 Mon Sep 17 00:00:00 2001
From: "scott%scott-macgregor.org"
 <scott%scott-macgregor.org@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 13 Aug 2003 19:07:11 +0000
Subject: [PATCH] Bug #190941 --> mailWindowOverlay.js uses obsolete outliner
 referenes. Thanks to Stephen Walker for porting this patch.

Standalone message window is a bit of a mess. There are several separators and menu items that need to be hidden. Add the appropriate ids to these items and then hide them.


git-svn-id: svn://10.0.0.236/trunk@145969 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/mail/base/content/mailWindowOverlay.xul | 10 ++++++----
 mozilla/mail/base/content/messageWindow.js      | 14 +++++++++++++-
 2 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/mozilla/mail/base/content/mailWindowOverlay.xul b/mozilla/mail/base/content/mailWindowOverlay.xul
index 2f866713d0f..faf59bee1ab 100644
--- a/mozilla/mail/base/content/mailWindowOverlay.xul
+++ b/mozilla/mail/base/content/mailWindowOverlay.xul
@@ -1060,7 +1060,7 @@
       <menuitem label="&findCmd.label;" key="key_find" accesskey="&findCmd.accesskey;" observes="cmd_find"/>
       <menuitem label="&findAgainCmd.label;" key="key_findAgain" accesskey="&findAgainCmd.accesskey;" observes="cmd_findAgain"/>
       <menuitem label="&findPrevCmd.label;"  key="key_findPrev"  accesskey="&findPrevCmd.accesskey;"  observes="cmd_findPrev"/>
-      <menuseparator/>
+      <menuseparator id="editPropertiesSeparator"/>
       <menuitem id="menu_properties" label="&folderPropsCmd.label;"
                 accesskey="&folderPropsCmd.accesskey;"
                 observes="cmd_properties"/>
@@ -1089,7 +1089,7 @@
               <menuitem id="menu_customizeToolbar" oncommand="CustomizeMailToolbar('mail-toolbox');" label="&customizeToolbar.label;" accesskey="&customizeToolbar.accesskey;"/>
             </menupopup>
           </menu>
-          <menu id="menu_MessagePane" label="&messagePaneLayoutStyle.label;" accesskey="&messagePaneLayoutStyle.accesskey;">
+          <menu id="menu_MessagePaneLayout" label="&messagePaneLayoutStyle.label;" accesskey="&messagePaneLayoutStyle.accesskey;">
             <menupopup onpopupshowing="InitViewLayoutStyleMenu()">
               <menuitem id="messagePaneClassic" type="radio" label="&messagePaneClassic.label;"
                         accesskey="&messagePaneClassic.accesskey;" oncommand="ChangeMailLayout(kClassicMailLayout);"/>
@@ -1102,7 +1102,9 @@
                         accesskey="&showMessageCmd.accesskey;" oncommand="MsgToggleMessagePane();"/>
             </menupopup>
           </menu>
-          <menuseparator/>
+          
+          <menuseparator id="viewSortMenuSeparator"/>
+
           <menu id="viewSortMenu" accesskey="&sortMenu.accesskey;" label="&sortMenu.label;">
             <menupopup onpopupshowing="InitViewSortByMenu()">
               <menuitem id="sortByDateMenuitem" type="radio" name="sortby" label="&sortByDateCmd.label;" accesskey="&sortByDateCmd.accesskey;" oncommand="MsgSortByDate()"/>
@@ -1220,7 +1222,7 @@
             </menupopup>
           </menu>
           <menuseparator id="goNextSeparator"/>
-          <menuitem label="&startPageCmd.label;" accesskey="&startPageCmd.accesskey;" oncommand="HideMessageHeaderPane();loadStartPage();"/>
+          <menuitem id="goStartPage" label="&startPageCmd.label;" accesskey="&startPageCmd.accesskey;" oncommand="HideMessageHeaderPane();loadStartPage();"/>
         </menupopup>
         </menu>
 
diff --git a/mozilla/mail/base/content/messageWindow.js b/mozilla/mail/base/content/messageWindow.js
index af414ba9e54..4e487954a4a 100644
--- a/mozilla/mail/base/content/messageWindow.js
+++ b/mozilla/mail/base/content/messageWindow.js
@@ -435,6 +435,10 @@ function HideMenus()
 	if (renameFolderMenu)
 		renameFolderMenu.setAttribute("hidden", "true");
 
+  var viewLayoutMenu = document.getElementById("menu_MessagePaneLayout");
+  if (viewLayoutMenu)
+    viewLayoutMenu.setAttribute("hidden", "true");
+
 	var viewMessagesMenu = document.getElementById('viewMessagesMenu');
 	if (viewMessagesMenu)
 		viewMessagesMenu.setAttribute("hidden", "true");
@@ -451,6 +455,10 @@ function HideMenus()
 	if (openMessageMenu)
 		openMessageMenu.setAttribute("hidden", "true");
 
+  var viewSortMenuSeparator = document.getElementById('viewSortMenuSeparator');
+  if (viewSortMenuSeparator)
+    viewSortMenuSeparator.setAttribute("hidden", "true");
+
 	var viewSortMenu = document.getElementById('viewSortMenu');
 	if (viewSortMenu)
 		viewSortMenu.setAttribute("hidden", "true");
@@ -459,6 +467,10 @@ function HideMenus()
 	if (emptryTrashMenu)
 		emptryTrashMenu.setAttribute("hidden", "true");
 
+  var menuPropertiesSeparator = document.getElementById("editPropertiesSeparator");
+  if (menuPropertiesSeparator)
+    menuPropertiesSeparator.setAttribute("hidden", "true");
+
 	var menuProperties = document.getElementById('menu_properties');
 	if (menuProperties)
 		menuProperties.setAttribute("hidden", "true");
@@ -471,7 +483,7 @@ function HideMenus()
 	if (trashSeparator)
 		trashSeparator.setAttribute("hidden", "true");
 
-	var goStartPageSeparator = document.getElementById('goStartPageSeparator');
+	var goStartPageSeparator = document.getElementById('goNextSeparator');
 	if (goStartPageSeparator)
 		goStartPageSeparator.hidden = true;