Closed Bug 75622 Opened 23 years ago Closed 20 years ago

3pane Mail Menu: Add/Change/Move/Delete menu items


(SeaMonkey :: MailNews: Message Display, defect, P3)



(Not tracked)



(Reporter: nbaca, Assigned: Stefan.Borggraefe)


(Depends on 1 open bug, )


(Keywords: access, Whiteboard: [adt3] [see comment 35])


(3 files, 1 obsolete file)

Build 2001-04-11-04: NT4
Still need to check Mac and Linux.

Overview: This bug summarizes the changes that should be made to the 3pane Mail  
Menu items. It includes mnemonic and accelerator issues as well.

1. Add
a. Menu Item
- File|Netscape Address Book Sync (w/mnemonic "k"),(Bugscape# 4613, Netscape 
- File|Page Setup (bug# 65871)
- View|Headers|Brief (bug# 60207)
- Search  (Bugscape# 4617, Netscape only)
- Message|Add Sender to Address Book (bug# 10860)
- Message|Add All to Address Book (bug# 10860)

b. Mnemonic
- File|New||Blank Page to Edit, needs "Ctrl+Shift+N"
- File| Attachments, need "A"
- File|Print Plus, need "l"

2. Change
a. Mnemonic
- File|Get New Message for , from "f" to "w"
- File|Offline, from "l" to "f"(so that Print Plus can have a mnemonic)

b. Accelerator
- Message|Mark|All Read (bug# 66681, from "A" to "Ctrl+Shift+C, now tbd)
Keywords: nsbeta1, ui
QA Contact: esther → nbaca
Addition to Change category:

2. Change
a. Mnemonic
- View|Messages| "Threads With Unread" and "Threaded" are both using "T". Change 
"Threads With Unread" to "h" to match the spec.
Depends on: 10860, 60207, 65871, 66681
Note: Print Plus is bugscape 2589
reassigning to ssu.
Assignee: sspitzer → ssu
3. Delete
a. Menu Item
- Edit Draft (Currently this appears disabled. Is there anyway to remove this 
menu item unless it is a message in the Draft folder? This effects the 3pane, 
Alt-3pane and Stand Alone window)
`Edit Message' is useful whether or not the message happens to be a draft. For 
example, if I send a message to a number of people, but forget one person, I 
can select the message in my Outbox and choose `Edit Message', removing the 
original recipients and replacing them with the one I forgot. I could use 
`Forward' instead, but `Edit Message' makes me look less absent-minded by 
hiding any evidence that I'd forgotten the recipient in the first place. :-)
To clarify point 3a, I was actually asking about the "Edit|Edit Draft" menu
item. Sorry for not being more clear.

I agree that the "Message|Edit Message As New" menu item should always be
available :-)
marking nsbeta1+ for 1b and all of 2.  I don't think we need to do 1a for this
Priority: -- → P3
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.1
Addition to Mnemonic category:
- File|Get New Messages for|Get All New Messages, needs "G".

jglick: Currently "Get All New Messages" displays "Ctrl+Shift+T" for an 
accelerator and it appears to be working. The spec states it should be 
"Ctrl+Shift+Y". I like "Ctrl+Shift+T" because it compliments Ctrl+T for the 
File|Get Messages menu item. Could the spec be updated with this change?
nbaca, yes it should be Ctrl+Shift+T
This spec has it correct:

I will fix the other spec. Thanks.
What happen to the View|Text Size menu item? It's completely gone.

Here is a summary of what needs to be present for the View|Text Size menus:

  - Smaller, needs "m" for mnemonic, Ctrl+-
  - Larger, needs "L" for mnemonic, Ctrl++
  - 50%, needs "5" for mnemonic
  - 75%, needs "7" for mnemonic
  - 90%, needs "9" for mnemonic
  - 100% (Original Size), needs "z" for mnemonic
  - 120%, needs "1" for mnemonic
  - 150%, needs "0" for mnemonic
  - 200%, needs "2" for mnemonic
  - Other, needs "O" for mnemonic
Additional Information on View|Text Size menu item
WinMe/Linux, 3pane:     OK 
WinMe/Linux, Alt-3pane: Not present
Mac, 3pane:             Not present, only double lines
Mac, Alt-3pane:         Not present, only double lines

Let me know if I should log seperate bugs on these specific issues.
It looks like nbaca's comments about the view | Text Size menu are covered in 78489.
moving to 0.9.2
Target Milestone: mozilla0.9.1 → mozilla0.9.2
Addition to Change category:
- View|Message should state "Message Pane" to make it more descriptive.

Note: The position of the "My Sidebar" menu item might change (bug# 79639, bug# 
80861). If "My Sidebar" is moved into the same position as the other toolbars 
then View|Message Pane should also move.
Addition to Mnemonic category:
- View|Messages|Ignored Threads, needs "I"

Addition to Change category:
- View|Messages submenu items are not listed in the same order as the spec.
Please refer to the spec or let me know if I should list what is expected in 
this bug.
moving to 0.9.3
Target Milestone: mozilla0.9.2 → mozilla0.9.3
Depends on: 85508, 85510, 85512
No longer depends on: 85512
Missed 0.9.3.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
moving to 0.9.5
Target Milestone: mozilla0.9.4 → mozilla0.9.5
I've fixed a seperate bug to track the View|Messages menu item inconsistencies.
 It's bug 97504.  I'll attach patches there.
QA Contact: nbaca → olgam
Branch build 2001-09-10: Windows 2000

Latest update:

1. Add
 a. Menu Item
    - File|Print Preview - Ctrl+Shift+P, need "Print Preview" with "V"  
underlined.  Also shortcut: Ctrl+Shift+P.  (Bug #17006)
 b. Mnemonics in the sub-menus:
    - File|Get New Messages for|Get All New Messages, need "G".
    - File|Offline|Work Offline, need "W".
    - View|Messages|Ignored Threads, need "I".
    - View|Character Coding|More  - needs mnemonic but it is not specified in 
    - when Newsgroup is selected: File|Get Next 500 News, need mnemonic - not  
"X" as Spec shows - since "X" is for Exit.

 c. Add Ellipsis
    - File|Offline|Offline Setting... (... - at the end) since it brings another 

2. Change
 a. Mnemonic
    - View|My Sidebar, from "b" to "M" according to Spec, but Navigator has "b".
There is conflict with View|Sort by, which also has "b".  Any way pressing on 
"b" performs no function.
    - Message|Ignore Thread, from "k" to "i" 
Keywords: nsbranch
Blocks: 99230
not an emojo stopper. However if sean is close to a fix then of course we'll
reconsider. But it isn't a stopper.
Keywords: nsbranchnsbranch-
Blocks: 76247
I checked Alt 3-pane window on Win 2000 and Linux.
The same issues as for 3-pane.
>Spec - when Newsgroup is selected: File|Get Next 500 News, need mnemonic - not  
>"X" as Spec shows - since "X" is for Exit.

"M" for mnemonic is available. Get Next 500 News _M_essages.
>2. Change
>a. Mnemonic
>- View|My Sidebar, from "b" to "M" according to Spec, but Navigator has "b".
>There is conflict with View|Sort by, which also has "b".  Any way pressing on 
>"b" performs no function.

This conflict happened when mozilla decided to remove "My" from "My Sidebar" but 
Netscape builds kept "My". Since Moz doesn't have "My", lets leave them with the 
"b" for mnemonic. 

Change "Sor_t_ by" to "t" for mnemonic instead of "b". Sound ok?
Jennifer, actually I have a fix for this exact bug.  The patch is attached to:
*** Bug 101425 has been marked as a duplicate of this bug. ***
moving to 0.9.6
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Blocks: 104166
moving to 1.0 and nominating
Keywords: nsbeta1
Whiteboard: [nsbeta1+]
Target Milestone: mozilla0.9.6 → mozilla1.0
Blocks: 107067
Keywords: nsbranch-
There is Bugscape bug 5716, which depends on this one.
BTW, I am going to eliminate many separate Bugzilla bugs, which have been logged
while ago.  For convenience purposes Ninoschka opened this summary bug for 3pane
window to keep all related issues together.  But there was no time to close
original ones.  Now it is the time.  Be prepared to see a lot of bugs listed as
Keywords: nsbeta1nsbeta1+
I thought adding the access key to the Communicator:WorkMode broadcaster would
fix this but it doesn't seem to work so instead the patch uses the menuitem
defined in utilityOverlay.xul instead. Subsequently I have noticed that
originally the menuitem didn't actually work, although this patch seems to fix
that as well.
Curioser and curioser...
Target Milestone: mozilla1.0 → mozilla0.9.9
Blocks: 122274
Keywords: nsbeta1+nsbeta1-
Target Milestone: mozilla0.9.9 → mozilla1.2
Depends on: 142152
Keywords: access
Blocks: 154249
I remove nsbeta1- keyword - it was for MachV.
Keywords: nsbeta1-
Now I add 'nsbeta1' keyword for Buffy. Sorry, it's faster to edit multiple bugs
at once than manually go to each and remove minus.
Keywords: nsbeta1
Keywords: nsbeta1+
Keywords: nsbeta1
Mail triage team: nsbeta1+/adt3
Whiteboard: [adt3]
Target Milestone: mozilla1.2alpha → mozilla1.4beta
Summarizing the various items mentioned already in this bug.

comment 0:
 - File|Netscape Address Book Sync  (w/ "k")  -- NO IDEA! DON'T HAVE NETSCAPE!
 - File|Page Setup  -- DONE
 - View|Headers|Brief  -- NOT DONE (bug 60207) HOWEVER... [note 1]
 - Search (toplevel menu) -- NO LONGER PART OF SPEC (Netscape only?)
 - Message|Add Sender/All to Address Book  -- NO LONGER PART OF SPEC 
       due to bug 10860[?]
 - File|New|Composer page shortcut -- DONE
 - File|Attachments mnemonic -- DONE
 - File|Print Plus -- NO LONGER PART OF SPEC (Netscape only?)

 - File|Get New Message For mnemonic -- DONE
 - File|Offline -- remains 'l' because 'print plus' no longer conflicts
 - Message|Mark|All Read -- DONE: is Ctrl+Shift+C, HOWEVER [note 2]

comment 1:
 - 'Threaded' mnemonic conflict -- DONE [note 3]

comment 4:
 - 'Edit Draft' -- NO LONGER PART OF SPEC

comment 8:
 - File|Get Messages For|Get All New Messages mnemonic -- NOT DONE

comment 10:
 - View|Size still reads View|Zoom (bug 139907) -- "Zoom" matches browser; 
   however, all submenu mnemonics are in place -- DONE

comment 11: per bug 78489, this problem is gone

comment 14:
 - View|Show/Hide|"Message Pane" -- DONE
 - View|Show/Hide|Sidebar -- NO LONGER PART OF SPEC

comment 15:  [note 4]
 - View|Threads|Ignored mnemonic -- DONE
 - View|Threads submenu items -- MATCHES SPEC [note 5]

comment 21:
 - File|Print Preview (including mnemonic) -- DONE;
        Print Preview shortcut  -- NOT PART OF SPEC
 - File|Offline|Work Offline mnemonic -- DONE
 - File|Get Next NN Messages mnemonic -- DONE
 - View|Character Coding|More mnemonic -- NOT DONE [note 6]
 - File|Offline|Offline Setting ellipsis -- NOT DONE
 - View|My Sidebar (mnemonic conflict) -- OK per comment 25:
     View|Sort By mnemonic -- DONE
 - Message|Ignore Thread shortcut -- MATCHES SPEC

[note 1] see bug 154712

[note 2] per bug 66681, current Ctrl+Shift+C may not best shortcut;
per bug 233182, this menu item might be better removed from the MESSAGE menu

[note 3] 'Threaded' on a different menu from 'Threads with Unread'; also, see 
bug 219620 [menu change, patch in progress]

[note 4] the old View|Messages is now View|Threads, spec not yet updated; this 
was bug 189543.

[note 5] this problem was spun off to bug 97504; however, that bug was never 
actually fixed, and the bug it was duped to fixed only the mnemonic issue -- the 
same basic patch as appears in this bug.

[note 6] Submenu for Character Coding is not specified in Mail Menus spec.
Presumably should follow browser's spec for equivalent menu item; browser *does* 
have mnemonic, on the 'M' in More.

Items still NOT DONE (repeating from above):
 - File|Get Messages For|Get All New Messages mnemonic -- NOT DONE
 - View|Character Coding|More mnemonic -- NOT DONE
 - File|Offline|Offline Setting ellipsis -- NOT DONE
Whiteboard: [adt3] → [adt3] [see comment 35]
Attached patch Fixes the remaining issues (obsolete) — Splinter Review
... so this bug can rest in peace.

Contrary to the findings in comment 35, the Character Encoding|More menu item
already has an access key. (What's strange with this submenu is that all its
menuitems also have access keys which are somehow not rendered. But this would
be another bug.)
Attachment #145244 - Flags: review?(timeless)
OS: Windows NT → All
Comment on attachment 145244 [details] [diff] [review]
Fixes the remaining issues

Get New Messages for>Get All New Messages
Get New Messages for>my account

I think that it should say:
Get New Messages for>All Accounts

Assignee: ssu0262 → Stefan.Borggraefe
I like that idea.
jshin fixed the _More menuitem in his patch in bug 55300.
Attachment #145244 - Flags: review?(timeless)
I like timeless' suggestion too, so here comes a new patch.
Attachment #145244 - Attachment is obsolete: true
Attachment #145552 - Flags: review?(timeless)
Attachment #145552 - Flags: review?(timeless) → review+
Attachment #145552 - Flags: superreview?(mscott)
Attachment #145552 - Flags: superreview?(mscott) → superreview+
Closed: 20 years ago
Resolution: --- → FIXED
Verified with 1.8a-0422.  Thanks, Stefan!  And thanks, Timeless, for the 
suggested change under Get New Messages For...  very nice.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.