Jared Wein [:jaws] (please needinfo? me) Reporter
	Description • 8 years ago

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>.

	Jared Wein [:jaws] (please needinfo? me) Reporter
	Comment 1 • 8 years ago

I ran mozregression and this has been broken since Undo was implemented,
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=802d87c77e76&tochange=a62bde1d6efe

	:Gijs (he/him) Assignee
	Comment 2 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/65552/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/65552/

	Jared Wein [:jaws] (please needinfo? me) Reporter
	Comment 3 • 8 years ago

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?

	:Gijs (he/him) Assignee
	Comment 4 • 8 years ago

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.

	Pulsebot
	Comment 5 • 8 years ago

Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3b35d43491f5
fix combined toolbar button styling after undo reset, r=jaws

	Carsten Book [:Tomcat]
	Comment 6 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/3b35d43491f5

	Md. Majedul islam
	Comment 7 • 8 years ago

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]

	Khalid Syfullah Zaman [:khalid32]
	Comment 8 • 8 years ago

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!

	Azmina
	Comment 9 • 8 years ago

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]