67540 - Go should be removed form single mail window

Status: VERIFIED INVALID

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Henrik Gemal]

If you double click on a mail and the single mail window is displayed the Go 
menuitem should be removed from the single mail window.
Reason: if you press it there is no way to get back to the mail withot closing 
the window.


mozilla build 2001020204

Comment 1

[[e-mail]]

Here's a patch. Someone else will have to ask for review.

Index: messageWindow.js
===================================================================
RCS file: /cvsroot/mozilla/mailnews/base/resources/content/messageWindow.js,v
retrieving revision 1.37
diff -u -r1.37 messageWindow.js
--- messageWindow.js    2001/07/02 22:03:07     1.37
+++ messageWindow.js    2001/07/09 16:33:59
@@ -263,6 +263,10 @@
        if(viewThreadedMenu)
                viewThreadedMenu.setAttribute("hidden", "true");

+       var goMenu = document.getElementById('goMenu');
+       if(goMenu)
+               goMenu.setAttribute("hidden", "true");
+
        var emptryTrashMenu = document.getElementById('menu_emptyTrash');
        if(emptryTrashMenu)
                emptryTrashMenu.setAttribute("hidden", "true");

Comment 2

[Håkan Waara]

I don't think the Go menu should be removed. It works fine here.

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

I think this is a dup, being discussed in that bug that mpt, gerv, german,
jglick and blake are on.

Comment 4

[(not reading, please use [e-mail] instead)]

we've decided that go (and search) are staying in the mailnews ui.

under the go menu are several useful and functional menu items for message
navigation.

next, next unread, etc.

since we're following the UI specs (http://www.mozilla.org/mailnews/specs), this
is invalid.

Comment 5

[Ninoschka Baca]

Verified.