Dennis Schubert [:denschub] Reporter
	Description • 7 years ago

After bug 1314091 landed, Private Windows look broken when opening them with a comapct theme selected. I've attached a screenshot from a private window opened with the dark comapct theme selected.

Note that this is not even the light compact theme, it's something... broken. Looks like a mixture between the compact theme and the default theme.

	Dennis Schubert [:denschub] Reporter
	Comment 1 • 7 years ago

For reference, this is how it should look like.

	Brian Grinstead [:bgrins]
	Comment 2 • 7 years ago

Thanks for the report, I can confirm the problem. I'll have a look this week.

	Brian Grinstead [:bgrins]
	Comment 3 • 7 years ago

It appears that lightweight themes don't get applied in private windows, so I suspect this is due to us changing the selectors to use -moz-lwtheme-darktext / -moz-lwtheme-brighttext instead of the custom attribute that we used to use: https://hg.mozilla.org/mozilla-central/rev/4333fc0c05eb#l12.2.

Gijs, any ideas here?  A couple thoughts would be (1) to allow these lightweight themes to apply to private windows (making a whitelist that contains just the compact themes) or (2) go back to using attributes for the css instead of the :-moz-lwtheme selectors.  I'd prefer 1 if it was feasible since that would let us keep the simpler CSS, and also close any potential gaps between normal and private windows that probably existed before but we didn't notice (due to styles like `#TabsToolbar:not(:-moz-lwtheme)` in browser.css being applied).

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

We should just enable lightweight themes for private windows. See bug 854126.

	:Gijs (he/him)
	Comment 5 • 7 years ago

(In reply to :Gijs from comment #4)
> We should just enable lightweight themes for private windows. See bug 854126.

(which has an extant patch but got stuck on OS X compat and then became sad and lonely - I wouldn't expect fixing it to be that difficult - famous last words, of course...)

	Dennis Schubert [:denschub] Reporter
	Comment 7 • 7 years ago

Sweet, works fine now. Thanks everyone!

	Paul Silaghi, QA [:pauly]
	Comment 8 • 7 years ago

Verified fixed Fx 53b1 Win 10, Ubuntu 14.04, OS X 10.11.