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)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 30
Tracking Status
firefox29 --- fixed
firefox30 --- fixed

People

(Reporter: mconley, Assigned: mconley)

References

(Blocks 1 open bug)

Details

(Whiteboard: [Australis:P3])

Attachments

(1 file)

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.
Whiteboard: [Australis:P2] → [Australis:P3]
Attached patch Patch v1Splinter Review
This... this looks pretty good, I must say. On Windows, anyway.
Assignee: nobody → mconley
Status: NEW → ASSIGNED
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)
Attachment #8387187 - Flags: review?(gijskruitbosch+bugs) → review+
remote:   https://hg.mozilla.org/integration/fx-team/rev/c23f0d27769c
Whiteboard: [Australis:P3] → [Australis:P3][fixed-in-fx-team]
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
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?
Attachment #8387187 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
QA Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: