Closed
Bug 1331276
Opened 8 years ago
Closed 8 years ago
Compact theme broken in Private Windows
Categories
(Firefox :: Theme, defect)
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.
Reporter | ||
Comment 1•8 years ago
|
||
For reference, this is how it should look like.
Comment 2•8 years ago
|
||
Thanks for the report, I can confirm the problem. I'll have a look this week.
status-firefox52:
--- → unaffected
Flags: needinfo?(bgrinstead)
Comment 3•8 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).
Flags: needinfo?(bgrinstead) → needinfo?(gijskruitbosch+bugs)
Comment 4•8 years ago
|
||
We should just enable lightweight themes for private windows. See bug 854126.
Depends on: 854126
Flags: needinfo?(gijskruitbosch+bugs)
Comment 5•8 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...)
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: fixed by bug 854126
Target Milestone: --- → Firefox 53
Updated•8 years ago
|
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•