hitting single-letter shortcuts should change focus from folder pane to thread pane

VERIFIED FIXED in mozilla0.9.8

Status

SeaMonkey
MailNews: Message Display
P2
normal
VERIFIED FIXED
16 years ago
13 years ago

People

(Reporter: sairuh (rarely reading bugmail), Assigned: Sean Su)

Tracking

({access})

Trunk
mozilla0.9.8
access

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 obsolete attachments)

(Reporter)

Description

16 years ago
spun off from bug 70097 [and bug 112477].

if focus is in the folder pane, hitting a single-letter shortcut should switch
focus to the thread pane.

examples:
1. click on a folder in the folder pane, or tab over to the folder pane and
select a folder with the up/down arrows. focus will be in the folder pane, of
course.
2. hit 'f' to display the first read/unread message in the thread pane. or, hit
'n' to display the first unread message in the thread pane.

actual results: the single-letter shortcut will display the message, but it will
*not* switch focus to the thread pane. focus currently remains in the folder
pane. you have to hit the Tab key to actually change focus [an extra step to
remember].
(Reporter)

Updated

16 years ago
Keywords: access
QA Contact: esther → olgam

Comment 1

16 years ago
reassigning to ssu.
Assignee: sspitzer → ssu
Keywords: nsbeta1+
Priority: -- → P2
Target Milestone: --- → mozilla0.9.7

Comment 2

16 years ago
Dup of bug 92693?  Bug bug is 'Hitting "next message" doesn't change selection 
focus to msg pane'.

Comment 3

16 years ago
I think they are related but not a dup.

Comment 4

16 years ago
Scott, would you please expand your comment #3?
It seems the same case with intentionally selected folder.
(Assignee)

Comment 5

16 years ago
Created attachment 61174 [details] [diff] [review]
patch to fix focus issue.

I've tested this with the standalone message window, and it does works fine. 
It does not set the focus back to mail&news window.

Comment 6

16 years ago
Comment on attachment 61174 [details] [diff] [review]
patch to fix focus issue.

r=varada
looks good to me.
Attachment #61174 - Flags: review+
(Assignee)

Comment 7

16 years ago
what I meant to say about the standalone message window is that it does not
break its functionality.
Status: NEW → ASSIGNED

Updated

16 years ago
Target Milestone: mozilla0.9.7 → mozilla0.9.8
it works from the stand alone msg window, but do you get any js errors /
warnings about SetFocusThreadPane() not being found to the console?

I'd think so, because it's defined in mail3PaneWindowCommands.js, which we
shouldn't be including into the stand alone msg window.  if we are, that's bad.

can you double check what's going on?

One question is, where should focus go in the std alone msg window after "n"?
I'd say into the message, so that space (for scroll down) works.  can you see if
it does that?  If not, maybe we can do this:

1) fix your patch to call SetFocusAfterGoNextMessage()
2) in mail3PaneWindowCommands.js. define SetFocusAfterGoNextMessage() to call
SetFocusThreadPane().  in messageWindow.js, define SetFocusAfterGoNextMessage()
to move focus to the right place (the message body).

(Assignee)

Comment 9

16 years ago
There are no error messages in the console when using the single letter shortcut
keys to advance to the next message.  It turns out that the standalone message
window does not call GoNextMessage().  It calls performNavigation() in
messageWindow.js.  

The focus in the standalone message window does go to the message pane after
using the single letter shortcut keys.
(Assignee)

Comment 10

16 years ago
adding seth to the cc: list.
(Assignee)

Comment 11

16 years ago
Created attachment 64179 [details] [diff] [review]
patch containing seth's recommendations

I didn't modify messageWindow.js because it is not affected, but I did create a
new function, SetFocusAfterGoNextMessage().
Attachment #61174 - Attachment is obsolete: true
(Assignee)

Comment 12

16 years ago
Comment on attachment 64179 [details] [diff] [review]
patch containing seth's recommendations

this bug will be fixed by the patch in bug 70097.
Attachment #64179 - Attachment is obsolete: true
(Assignee)

Updated

16 years ago
Depends on: 70097
(Assignee)

Comment 13

16 years ago
patch for bug 70097 got checked in.  marking this fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 14

16 years ago
Verified on Win2K, Linux, Mac OSX - trunk build 01-29-2002.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.