Closed Bug 1331276 Opened 8 years ago Closed 8 years ago

Compact theme broken in Private Windows

Categories

(Firefox :: Theme, defect)

53 Branch
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 53
Tracking Status
firefox52 --- unaffected
firefox53 --- verified

People

(Reporter: denschub, Unassigned)

References

Details

(Keywords: regression, Whiteboard: fixed by bug 854126)

Attachments

(2 files)

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.
For reference, this is how it should look like.
Thanks for the report, I can confirm the problem. I'll have a look this week.
Flags: needinfo?(bgrinstead)
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).
Flags: needinfo?(bgrinstead) → needinfo?(gijskruitbosch+bugs)
We should just enable lightweight themes for private windows. See bug 854126.
Depends on: 854126
Flags: needinfo?(gijskruitbosch+bugs)
(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...)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: fixed by bug 854126
Target Milestone: --- → Firefox 53
Sweet, works fine now. Thanks everyone!
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Verified fixed Fx 53b1 Win 10, Ubuntu 14.04, OS X 10.11.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: