Closed
Bug 979054
Opened 10 years ago
Closed 10 years ago
Try adding toolbar edge padding before customization transition starts, not after.
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
RESOLVED
FIXED
Firefox 30
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 1 open bug)
Details
(Whiteboard: [Australis:P3])
Attachments
(1 file)
954 bytes,
patch
|
Gijs
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
We add some padding to the start and end of the toolbar customization targets to make it easier to add things at the start and end of them. That padding is currently being added at the end of the customize mode transition. Adding it to the beginning should make the transition finish less jarring.
Assignee | ||
Updated•10 years ago
|
Whiteboard: [Australis:P2] → [Australis:P3]
Assignee | ||
Comment 1•10 years ago
|
||
This... this looks pretty good, I must say. On Windows, anyway.
Assignee: nobody → mconley
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8387187 [details] [diff] [review] Patch v1 phlsa likes this, and so do I. I think this makes things a lot smoother.
Attachment #8387187 -
Flags: review?(gijskruitbosch+bugs)
Updated•10 years ago
|
Attachment #8387187 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 3•10 years ago
|
||
remote: https://hg.mozilla.org/integration/fx-team/rev/c23f0d27769c
Whiteboard: [Australis:P3] → [Australis:P3][fixed-in-fx-team]
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c23f0d27769c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P3][fixed-in-fx-team] → [Australis:P3]
Target Milestone: --- → Firefox 30
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8387187 [details] [diff] [review] Patch v1 [Approval Request Comment] Bug caused by (feature/regressing bug #): Australis! User impact if declined: A slightly jankier customize mode transition - there'll be more of a "bump" when the transition ends. Testing completed (on m-c, etc.): Lots of local testing. Risk to taking this patch (and alternatives if risky): Very low. String or IDL/UUID changes made by this patch: None.
Attachment #8387187 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Updated•10 years ago
|
Attachment #8387187 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c888d74d3963
Updated•10 years ago
|
QA Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•