Closed
Bug 112577
Opened 23 years ago
Closed 23 years ago
message pane doesn't show focus ring
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.8
People
(Reporter: neil, Assigned: hewitt)
References
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
patch
|
timeless
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
For some reason when I click on the message pane the focus ring doesn't appear, possibly because the commandDispatcher is returning null for the focusedElement.
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Updated•23 years ago
|
Attachment #59674 -
Flags: review+
Reporter | ||
Comment 2•23 years ago
|
||
CCing bryner because shaver tells me he knows about the command dispatcher.
We need more details when there is no focus in Thread pane. There are dif bugs regarding this.
From duplicate I see "the message display window". You refer to "message pane". It looks like that you both talk about Message Body area. It's covered in bug 109193: Focus ring is only in the Thread pane, not in Folder or Message Body pane. Do you mind if I mark it as duplicate as well?
Reporter | ||
Comment 7•23 years ago
|
||
I have no problems with the focus in the folder pane, so it could be mac-specific.
May we clarify the issue here? Now you say "no problems with the focus in the folder pane". Also why Mac, bug shows Win 95.
Comment 10•23 years ago
|
||
Thanks for raising the issue! *** This bug has been marked as a duplicate of 109193 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 12•23 years ago
|
||
Firstly, you shouldn't resolve and verify the same bug. Secondly, my original point is that this report only applies to the message body area, not to the Mac folder pane. My Windows folder pane focuses perfectly.
Status: VERIFIED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Comment 13•23 years ago
|
||
this one has been bothering me too
Assignee: sspitzer → hewitt
Status: REOPENED → NEW
Target Milestone: --- → mozilla0.9.8
Assignee | ||
Comment 14•23 years ago
|
||
This patch does a bit more than neil's earlier patch. It cleans up some incredibly ugly code in FocusRingUpdate_Mail and does a more explicit check to see if the message pane is focused.
Reporter | ||
Comment 15•23 years ago
|
||
Comment on attachment 61006 [details] [diff] [review] patch I don't understand: 1. Why do you check the current focused element is the thread outliner, folder outliner or message pane before setting the focus ring, becuase it must always be one of the three. Or, you could just check for the message pane and use :focus rules for the thread and folder panes. 2. Why don't you check for the message pane. If the attachment tree has the focus then WhichPaneHasFocus should return the message pane. 3. focusedWindow looks like a cheap operation so it should be done first.
Assignee | ||
Comment 16•23 years ago
|
||
Alas, there is one other thing in the mail window that can get focus: the sidebar, which is why I check if each of the three panes has the focus.
Status: NEW → ASSIGNED
Comment 17•23 years ago
|
||
Updating multiple bugs. This is a valid UI issue. Would be nice to have it fixed if time allows.
Reporter | ||
Comment 18•23 years ago
|
||
if (!currentFocusedElement) return; takes care of the sidebar, because it is never returned from WhichPaneHasFocus(). See also the combined patch attached to bug 102711.
Assignee | ||
Comment 19•23 years ago
|
||
fix checked in, updated after comments from neil
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 20•23 years ago
|
||
var gSearchInput; missing from msgMail3PaneWindow.js (patch is in bug 102711).
Comment 22•23 years ago
|
||
Verified on today's and yesterday build 2002-01-03. Win 2K, Linux, Mac OSX.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•