Closed Bug 591120 Opened 14 years ago Closed 14 years ago

Change flow of setup wizard

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: philikon, Assigned: philikon)

References

Details

(Whiteboard: [strings])

Attachments

(1 file, 3 obsolete files)

Mostly involves moving engine and first sync settings into separate Sync Options dialog.
Component: Sync → Firefox UI
QA Contact: sync → firefox
The Sync Options dialog looks like another wizard page in the mockup, but I think it makes more sense to implement it was a dialog. It's clearly modal and it will be reused for Reset Sync (yes, we reuse the wizard there right now, but that always felt weird to me).
Whiteboard: [strings]
The only problem with making this a dialog is that Reset Sync would lose the confirmation page. But we don't really have it in the wizard anymore, either (unless you count the Setup Complete page as such). Thoughts?
Attached patch v1 (obsolete) (deleted) — Splinter Review
This changes the setup flow to match Alex's mockups. Some notes: * This does change strings where appropriate, but it might not have covered all. We should probably revise any outstanding strings in a separate bug. * While the size of the wizard and the styling of certain elements are changed, this isn't meant to include any significant styling changes. See bug 591122 and bug 592839 for those.
Assignee: nobody → philipp
Attachment #471390 - Flags: review?(mconnor)
Attachment #471390 - Flags: feedback?(faaborg)
Attached patch v1.1 (obsolete) (deleted) — Splinter Review
Small rev on v1: Also switch over the accesskey when relabeling the buttons (for going into the Sync Options part of the wizard).
Attachment #471390 - Attachment is obsolete: true
Attachment #471537 - Flags: review?(mconnor)
Attachment #471537 - Flags: feedback?(faaborg)
Attachment #471390 - Flags: review?(mconnor)
Attachment #471390 - Flags: feedback?(faaborg)
Attached patch v2 (obsolete) (deleted) — Splinter Review
Incorporate faaborg's feedback: * Rename the "Create you Firefox Sync Account" page to "Account Details" as per mockups. * Rename and move the "Server" form field as per mockups. * Check email validity when the textbox loses focus, not on every character. * Display captcha error messages inline instead of prompting an alert dialog.
Attachment #471537 - Attachment is obsolete: true
Attachment #471878 - Flags: review?(mconnor)
Attachment #471537 - Flags: review?(mconnor)
Attachment #471537 - Flags: feedback?(faaborg)
Attached patch v2.1 (deleted) — Splinter Review
One more feedback from faaborg: * Use syncBrand.shortName (Sync) instead of syncBrand.fullName (Firefox Sync) on Sync Options page.
Attachment #471878 - Attachment is obsolete: true
Attachment #471897 - Flags: review?(mconnor)
Attachment #471878 - Flags: review?(mconnor)
Attachment #471897 - Flags: review?(mconnor) → review+
Blocks: 530818
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Blocks: 590805
Verified. Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b7pre) Gecko/20100924 Firefox/4.0b7pre
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: