Closed Bug 564095 Opened 14 years ago Closed 14 years ago

wizard cleanup

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: mconnor, Assigned: mconnor)

Details

Attachments

(1 file)

Attached patch v0.6Splinter Review
We know it's kind of ugly, at least we can make it look like something kinda decent and shiny to go to with the rebranding.  There's also miscellaneous bugfixes in here, I could split them out if need be.

http://grab.by/4ej4 is an example of what this would look like (plus branding changes)
Flags: blocking-weave1.3+
Target Milestone: 1.3 → 1.3b5
Comment on attachment 443789 [details] [diff] [review]
v0.6

Let's just get this reviewed, we can do more later if we find issues.
Attachment #443789 - Flags: review?(edilee)
Comment on attachment 443789 [details] [diff] [review]
v0.6

>+++ b/source/chrome/content/preferences/fx-setup.xul
>+          <hbox>
>+            <image class="statusIcon"/>
>+            <label class="status" value=" "/>
>+          </hbox>
Just checking, these hbox/images+labels dont need to worry about stretching the image?

Also do we need to change the prefpane? I guess the user would only see it if s/he opens prefs manually because autorun will open the wizard.
Attachment #443789 - Flags: review?(edilee) → review+
http://hg.mozilla.org/labs/weave/rev/0c3f4f218986
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
verified with 1.3b5
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: