Closed
Bug 486333
Opened 15 years ago
Closed 14 years ago
Allow engine selection when setting up additional machines
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
VERIFIED
FIXED
1.3b4
People
(Reporter: Mardak, Assigned: mconnor)
References
Details
(Keywords: uiwanted)
Attachments
(1 file)
1.81 KB,
patch
|
Mardak
:
review+
|
Details | Diff | Splinter Review |
Bug 480057 added a way to pick the direction to sync but it applied to every engine. We should populate the drop-down with enabled engines.
Updated•15 years ago
|
Target Milestone: -- → 1.0
Assignee | ||
Updated•15 years ago
|
Component: Weave → General
Product: Mozilla Labs → Weave
Version: Trunk → unspecified
Updated•15 years ago
|
QA Contact: weave → general
Updated•15 years ago
|
Priority: -- → P2
Assignee | ||
Updated•14 years ago
|
Component: General → Firefox UI
QA Contact: general → firefox
Target Milestone: 1.0 → 1.3
Assignee | ||
Comment 1•14 years ago
|
||
Since we close the prefwindow once Weave is set up, there's no chance for the user to fix this.
Flags: blocking-weave1.3+
Keywords: uiwanted
Priority: P2 → --
Summary: Allow engine selection when picking sync → Allow engine selection when setting up additional machines
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → paul
Comment 2•14 years ago
|
||
We will infer settings from info/collections and surface them on additional computer setup.
Assignee | ||
Updated•14 years ago
|
Assignee: paul → mconnor
Whiteboard: [b2]
Assignee | ||
Updated•14 years ago
|
Whiteboard: [b2]
Target Milestone: 1.3 → 1.3b3
Assignee | ||
Updated•14 years ago
|
Target Milestone: 1.3b3 → 1.3b4
Assignee | ||
Comment 3•14 years ago
|
||
Inserts the prefs screen after the merge choice. We'll spin off the "pre-fill these with the values inferred from the server" bit to a new bug.
Attachment #443151 -
Flags: review?(edilee)
Reporter | ||
Comment 4•14 years ago
|
||
Comment on attachment 443151 [details] [diff] [review] v1 Kinda unrelated, but onPageShow has "case 9:" instead of SETUP_SUCCESS_PAGE. Made it slightly trickier to make sure that the success page didn't have a back button for new account and existing account paths.
Attachment #443151 -
Flags: review?(edilee) → review+
Reporter | ||
Comment 5•14 years ago
|
||
http://hg.mozilla.org/labs/weave/rev/dc53dc51bdc3
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•14 years ago
|
||
Fixed the case 9 bit, should have caught that before. filed Bug 563459 for inferring initial values.
Updated•6 years ago
|
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.
Description
•