51800 - Accelerators Mail must have for beta3

Status: VERIFIED WORKSFORME

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[Lakespur Roca]

These are the Accelerators that are Must Haves for Mail.
Accelerators that are duplicated in all parts of the application have a
different bug. (51418).

Alt - Alt on Windows and Linux, Option on MacOS (unless stated otherwise)
Ctrl - Ctrl on Windows and Linux, Cmd on MacOS (unless stated otherwise)

Accelerator             Behavior
Ctrl + F                Find in Message
Ctrl + K                Check Spelling
Ctrl + L                Forward
Ctrl + O                Open File
Ctrl + R                Reply
Ctrl + S                Save
Ctrl +  + or =          Increase indent
Ctrl + -                Decrease indent

Other Accelerators
Ctrl + Enter            Send Mail

N                       Go Next Unread Message
P                       Go Previous Unread Message
T                       Go Next Unread Thread

Enter                   Open Message

Tabb and the arrow keys work for the main mail window already they are just hard
to use since the visual indication of focus for the active pane is not working yet.

Codeing comentary is welcome here. Other comentary please see the tracking bug.

Comment 1

[sairuh (rarely reading bugmail)]

not sure who owns this on the mailnews side...

Comment 2

[sairuh (rarely reading bugmail)]

*** Bug 51801 has been marked as a duplicate of this bug. ***

Comment 3

[scottputterman]

With one week left it's a little late to be filing must have bugs for something 
like accelerators!

However, in my build from today it looks like we have everything working except 
for ENTER in the tree widget.  What is Open File in mailnews?  Currently Ctrl-O 
stands for "open message in new window" which works.

So, I'm going to mark WFM since Varada already has a bug for ENTER.

Comment 4

[lchiang]

We already have bugs for mail for accelerator keys which have been filed 
separately.  Like Scott says, most of these are functional.   Nbaca - can you 
just reference the other bug numbers here for Lake?  thanks.

Comment 5

[Lakespur Roca]

Thanks guys. I knew that most of these were already in. (Thanks to the diligence 
and perspacity of the Mail team!) but this bug was needed as a check list for QA 
and PDT and others to understand what is this release what is overall and what 
is to be implimented later. 
Also The Ctrl O was a mistake. When editing this bug I put the Browser Ctrl O 
function in where it should have been open  message.

Thanks for not getting bent out of shape.  :-)

Comment 6

[Ninoschka Baca]

Bug# 48985 covers 3pane Accelerators.