Closed
Bug 1089812
Opened 10 years ago
Closed 10 years ago
Implement updated In-content pref secondary dialogs
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
Tracking | Status | |
---|---|---|
firefox36 | --- | verified |
People
(Reporter: mmaslaney, Assigned: ntim)
References
Details
Attachments
(1 file, 4 obsolete files)
3.77 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
Implement the updated In-content pref secondary dialogs. Mocks: http://people.mozilla.org/~mmaslaney/incontent/Preferences-dialog-types.png Spec: http://people.mozilla.org/~mmaslaney/incontent/Preferences-dialog-types-spec.png
Updated•10 years ago
|
Blocks: ship-incontent-prefs
Updated•10 years ago
|
Flags: firefox-backlog+
Assignee | ||
Comment 2•10 years ago
|
||
This patch : - Updates the dialog styling - Updates the tree styling (both to match the dialog spec) - Note that I put the tree styling inside the shared file, since it's gonna be used for other in content pages
Assignee | ||
Comment 3•10 years ago
|
||
Addressed feedback by mmaslaney.
Attachment #8520712 -
Attachment is obsolete: true
Attachment #8520712 -
Flags: review?(jaws)
Attachment #8520742 -
Flags: review?(jaws)
Assignee | ||
Comment 4•10 years ago
|
||
Removed the tree and listbox changes. Moved them to bug 1087618
Attachment #8520742 -
Attachment is obsolete: true
Attachment #8520742 -
Flags: review?(jaws)
Attachment #8520832 -
Flags: review?(jaws)
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8520832 [details] [diff] [review] Patch v3 Review of attachment 8520832 [details] [diff] [review]: ----------------------------------------------------------------- Need to rebase this on top of bug 1062127
Attachment #8520832 -
Flags: review?(jaws)
Assignee | ||
Comment 6•10 years ago
|
||
Rebased.
Attachment #8520832 -
Attachment is obsolete: true
Attachment #8521389 -
Flags: review?(dao)
Comment 7•10 years ago
|
||
Comment on attachment 8521389 [details] [diff] [review] Patch v4 >-} >+} >\ No newline at end of file Please fix this throughout the patch.
Attachment #8521389 -
Flags: review?(dao)
Assignee | ||
Comment 8•10 years ago
|
||
Restored newlines at end of files
Attachment #8521389 -
Attachment is obsolete: true
Attachment #8521484 -
Flags: review?(dao)
Updated•10 years ago
|
Attachment #8521484 -
Flags: review?(dao) → review+
Assignee | ||
Updated•10 years ago
|
Whiteboard: checkin-needed
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8ce7d46f1f99
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: checkin-needed
Target Milestone: --- → Firefox 36
Updated•10 years ago
|
Iteration: --- → 36.3
Flags: qe-verify?
Updated•9 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: camelia.badau
Comment 10•9 years ago
|
||
Verified fixed on Windows 7 64bit, Ubuntu 13.10 32bit and Mac OSX 10.9.5 using Firefox 36 Beta 3 (buildID: 20150122214638).
Status: RESOLVED → VERIFIED
status-firefox36:
--- → verified
You need to log in
before you can comment on or make changes to this bug.
Description
•