Closed Bug 94800 Opened 23 years ago Closed 22 years ago

Linux,Mac: Search/Filters UI: Needs mouse click to focus for tab/keyboard access

Categories

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

x86
Other
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.4beta

People

(Reporter: laurel, Assigned: ssu0262)

References

Details

(Keywords: access, Whiteboard: [adt3] seek r=?)

Attachments

(1 file)

Using aug10 commercial trunk build Related bug 40492 Problem applies to Linux and Mac. Win98 is OK. Tabbing to/from fields is generally working in the search messages and filter messages UIs. However, on linux and mac you can't get keyboard access until you first mouse click to get the window focus. On windows, I'm able to launch the dialog and hit Tab which sets focus in the search/filter window on the first field. On the other platforms Tab (even repeatedly) doesn't get me into focus anywhere in the respective ui until I mouse click in the window. Steps: 1. Launch to mail window, Search|Search Messages or Edit|Message Filters. 2. Hit Tab key. Notice nothing appears to be selected/focused. Try using arrow keys to see if anything is selected. Result: On win98, focus is easily spotted on the first field/scope dropdown. Result: On mac and linux, nothing happens, nothing focused. 3. Hit Tab again, notice no change, no focus in mac and linux. 4. In mac and linux, mouse click to focus in window (anywhere), then tabbing to and from fields works.
Keywords: access
QA Contact: esther → laurel
*** Bug 112340 has been marked as a duplicate of this bug. ***
still exists, may also overlap with other ui bugs.
Keywords: nsbeta1
Status: NEW → ASSIGNED
Keywords: nsbeta1nsbeta1+
Priority: -- → P2
Is this really a nsbeta1+, P2? If yes, then we need to try and schedule before M1.0.
ssu, feel free to take this since it is an access bug.
on second thought, I'll just give this one to ssu.
Assignee: sspitzer → ssu
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
Discussed in 2/25 bug mtg w Mktng, PjM and engineering. Decision was to move to 1.2 and nsbeta minus this bug.
Blocks: 122274
Keywords: nsbeta1+nsbeta1-
Target Milestone: mozilla1.0 → mozilla1.2
*** Bug 148065 has been marked as a duplicate of this bug. ***
Attached patch the patch file of the focus bug (deleted) — Splinter Review
Actually, I create the patch for bug 148065. As 148065 is treated at a duplication of this bug. I put my patch here. The situation here is a little like the "Find in this page..." window. It will select the contend first then give focus on it. So I add one line to explicit call the focus.
See also bug 68891, "mail search window should focus textbox automatically" (all platforms).
Whiteboard: seek r=?
Blocks: 154249
nominating for next release
Keywords: nsbeta1-nsbeta1
Mail triage team: nsbeta1+/adt3
Keywords: nsbeta1nsbeta1+
Whiteboard: seek r=? → [adt3] seek r=?
Comment on attachment 85698 [details] [diff] [review] the patch file of the focus bug Looks good to me.
Attachment #85698 - Flags: review+
Target Milestone: mozilla1.2alpha → mozilla1.4beta
Comment on attachment 85698 [details] [diff] [review] the patch file of the focus bug requesting sr= for Robin's patch since it's trivial.
Attachment #85698 - Flags: superreview?(sspitzer)
Comment on attachment 85698 [details] [diff] [review] the patch file of the focus bug sr=sspitzer ssu, if you check in, can you add a comment above the focus() call pointing back to this bug? since this isn't needed on win32, the comment will help ensure someone doesn't remove the workaround.
Attachment #85698 - Flags: superreview?(sspitzer) → superreview+
I'm not sure if Robin Lu has access to cvs, so I checked the patch in for him. Comment request by Seth also added to patch. closing bug as fixed.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Using trunk builds 20030303 on linux and macosx this works like Windows now, verfied.
Status: RESOLVED → VERIFIED
QA Contact: laurel → esther
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: