Closed Bug 558480 Opened 14 years ago Closed 14 years ago

Account Setup dialog steals focus from reading the TOC website

Categories

(Firefox :: Sync, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: tchung, Assigned: mconnor)

Details

(Keywords: polish, Whiteboard: [weave1.2b5])

Attachments

(1 file, 1 obsolete file)

Attached image weave TOC screenshot (obsolete) —
When selecting the weave terms of condition on the new account creation dialog, the window steals focus and does not allow the user to navigate to the browser to actually view the TOC.  The only way they can do that is to cancel their registration.  

I suggest defocusing the window and allowing the user to navigate back to the TOC window after they have control again.  

See screenshot

Repro:
1) install weave 1.2b5
2) create a new weave account
3) at the new account page, click the "Terms of Service" link
4) notice the TOC appears in the browser window behind the TOC window, but the user cannot unfocus from the TOC dialog unless they proceed with action.  This is useless if the user cannot view the TOC before the accept it.

Expected:
- new user account dialog should not have a locked focus, and thus allow the user to view the TOC

Actual:
- unable to get to TOC page without cancelling or next dialog
Flags: blocking-weave1.3?
Flags: blocking-weave1.2?
Whiteboard: [weave1.2b5]
Summary: weave TOC dialog steals focus from reading the TOC website → Account Setup dialog steals focus from reading the TOC website
Keywords: polish
I hate modal prefwindows so much.
Assignee: nobody → mconnor
Target Milestone: --- → 1.3
Flags: blocking-weave1.3?
Flags: blocking-weave1.3+
Flags: blocking-weave1.2?
Probably also true for the reset password link.
Whiteboard: [weave1.2b5] → [weave1.2b5][b2]
Whiteboard: [weave1.2b5][b2] → [weave1.2b5]
Target Milestone: 1.3 → 1.3b3
Attached patch v1Splinter Review
This gets moved and cleaned up better in bug 562183, but this is the specific fix for that bug.
Attachment #438201 - Attachment is obsolete: true
Attachment #441948 - Flags: review?
Attachment #441948 - Flags: review? → review+
http://hg.mozilla.org/labs/weave/rev/f1884c55a3ca
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
verified with 1.3b6
Status: RESOLVED → VERIFIED
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: