arni2033 Reporter
	Description • 8 years ago

>>>   My Info:   Win7_64, Nightly 52, 32bit, ID 20161028030204 (2016-10-28)
Should think more global before "improving" anything

>>>
STR_1:
1. Open devtools -> memory

AR:  Clear button is disabled
ER:  Clear button should be enabled, just like in any other tool


STR_2:  (reference of good behavior)
1. Open devtools -> performance

AR:  Clear button is enabled, just as expected

Note:
 Obviously, there're 2 ways to make UI consistent, either ER of STR_1, which I consider as good UI, or
"clear button should be disabled in all tools if there's nothing to clear", which I consider as bad UI
 The easy explanation is that there's nothing bad with clear button being enabled or not, but if you
 change anything, it'll require additional effort from user to get used to new UI you're imposing.
 If you aim at good UI, you'll only have to fix one line of code (patch from bug 1301815).
 
This is regression from bug 1301815. Regression range:
> https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=d8bbf1a3957fd25ff24bfee51331c150b154cc39&tochange=2c8e364b56e11e53b2aba48653b0ed77224a69df