925338 - Remove obsolete defaultmode, lockmode, defaulticonsize, lockiconsize attributes

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[Dão Gottwald [:dao]]

Attachment: patch

Comment 1

[:Gijs (out for now; he/him)]

Comment on attachment 815360 [details] [diff] [review]
patch

r=me, but can you file a followup to purge mode="text" / mode="full" selectors from the CSS, too? We should do this only when we've merged to m-c though, because otherwise we'll just end up merging in more... :-(

Comment 2

[Dão Gottwald [:dao]]

http://hg.mozilla.org/projects/ux/rev/aad230472afb

(In reply to :Gijs Kruitbosch from comment #1)
> We should do this only when we've merged to m-c
> though, because otherwise we'll just end up merging in more... :-(

I'm not sure what this means...

Comment 3

[:Gijs (out for now; he/him)]

(In reply to Dão Gottwald [:dao] from comment #2)
> http://hg.mozilla.org/projects/ux/rev/aad230472afb
> 
> (In reply to :Gijs Kruitbosch from comment #1)
> > We should do this only when we've merged to m-c
> > though, because otherwise we'll just end up merging in more... :-(
> 
> I'm not sure what this means...

So, AIUI, we should get rid of mode="full" or mode="text" selectors, because they're now useless, as we're always using mode="icons". However, we *did* get rid of them - but because we merge stuff from m-c, I noticed that there are still/again selectors in the UX tree which use these (that presumably got merged in from m-c). As they don't actually hurt much apart from the infinitesimal amount of extra time spent checking if those selectors match, we should probably wait with doing a last purge of the never-matching selectors until we've merged to m-c.

Comment 4

[:Gijs (out for now; he/him)]

https://hg.mozilla.org/mozilla-central/rev/aad230472afb