Closed Bug 688168 Opened 8 years ago Closed 8 years ago

[TABLETUI] Make the sync dialog pretty

Categories

(Firefox for Android Graveyard :: General, defect)

Firefox 9
All
Android
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 9

People

(Reporter: ibarlow, Assigned: mbrubeck)

References

Details

Attachments

(2 files)

Attached image Sync dialog mockup
No description provided.
Should build on the work done in bug 681643
Blocks: 655762
OS: Mac OS X → Android
Hardware: x86 → All
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
The increased header spacing and thicker separator line also apply to the "Edit Bookmark" dialog, which shares its structure and styles with the sync dialog.  I think it's an improvement there too.
Attachment #562180 - Flags: review?(wjohnston)
Comment on attachment 562180 [details] [diff] [review]
patch

Review of attachment 562180 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/themes/core/honeycomb/browser.css
@@ +1726,5 @@
>  }
>  
>  /* Sync setup ------------------------------------------------------------- */
>  .content-dialog > .prompt-title {
> +  margin: @margin_large@

forgot the ;
Attachment #562180 - Flags: review?(wjohnston) → review+
(In reply to Wesley Johnston (:wesj) from comment #3)
> forgot the ;

Fixed, thanks.

https://hg.mozilla.org/integration/mozilla-inbound/rev/9de2292c7a8d
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/9de2292c7a8d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 9
Mozilla/5.0 (Android; Linux armv7l; rv:9.0a1) Gecko/20110925 Firefox/9.0a1 Fennec/9.0a1
Status: RESOLVED → VERIFIED
Depends on: 690870
You need to log in before you can comment on or make changes to this bug.