Closed Bug 1287980 Opened 3 years ago Closed 3 years ago

Combined toolbar buttons get huge after Restore Defaults -> Undo

Categories

(Firefox :: Toolbars and Customization, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 50
Tracking Status
firefox50 --- verified
firefox51 --- verified

People

(Reporter: jaws, Assigned: Gijs)

References

Details

Attachments

(1 file)

STR:
Move the Cut/Copy/Paste buttons to the toolbar
Enter customization mode
Click Restore Defaults
Click Undo

ER:
The cut/copy/paste buttons should be normal size.

AR:
The buttons are supersized.

This is because the Undo operation puts the buttons back in the toolbar but it doesn't add the .toolbarbutton-1 class to the nested <xul:toolbarbutton> within the <xul:toolbaritem>.
I ran mozregression and this has been broken since Undo was implemented,
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=802d87c77e76&tochange=a62bde1d6efe
Depends on: 970511
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment on attachment 8772818 [details]
Bug 1287980 - fix combined toolbar button styling after undo reset,

https://reviewboard.mozilla.org/r/65552/#review63310

::: browser/base/content/browser.xul
(Diff revision 1)
> -    <!--
> -           CAVEAT EMPTOR
> -           Should you need to add items to the toolbar here, make sure to also add them
> -           to the default placements of buttons in CustomizableUI.jsm, so the
> -           customization code doesn't get confused.
> -      -->

Why don't we want this warning anymore?
Attachment #8772818 - Flags: review?(jaws) → review+
https://reviewboard.mozilla.org/r/65552/#review63310

> Why don't we want this warning anymore?

It was originally added to explain the duplication between `defaultset` and CUI.jsm. Defaultset is gone now, and people are pretty used to adding things to/via CUI.jsm these days, so I don't think it's useful to keep it.
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3b35d43491f5
fix combined toolbar button styling after undo reset, r=jaws
https://hg.mozilla.org/mozilla-central/rev/3b35d43491f5
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
I have reproduced this bug with Firefox nightly 50.0a1 (2016-07-19) on Windows 10, 64 Bit.

The bug's fix is now verified on latest nightly 51.0a1.

Build ID 	20160824030337
User Agent 	Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0

[bugday-20160824]
I have reproduced this bug on Nightly 50.0a1 (2016-07-19) (Build ID: 20160719030224) on Linux, 64 bit!

The bug's fix is now verified on Latest Nightly 51.0a1!

Build ID: 20160824030337
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0
OS: Ubuntu 16.04, Linux 4.4.0-34-generic

As this bug’s fix is also verified on Windows 10, 64 bit (comment 7), I am marking this as verified!
Status: RESOLVED → VERIFIED
QA Whiteboard: [bugday-20160824]
I have reproduced this bug with Nightly 50.0a1 (2016-07-19) on Windows 8.1 (64 bit).

This bug's fix is verified on Latest Aurora 50.0a2.
Build ID : 20160826004001
User Agent : Mozilla/5.0 (Windows NT 6.3; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0

[testday-20160826]
You need to log in before you can comment on or make changes to this bug.