Closed
Bug 1417396
Opened 7 years ago
Closed 7 years ago
MacOS titlebar renders differently in default and lightweight themes.
Categories
(Firefox :: Theme, defect, P5)
Tracking
()
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox57 | --- | wontfix |
firefox58 | --- | wontfix |
firefox59 | --- | wontfix |
firefox60 | --- | wontfix |
firefox61 | --- | fix-optional |
People
(Reporter: u595769, Unassigned)
References
Details
(Keywords: regression)
Attachments
(2 files)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20171112125346
Steps to reproduce:
1. Enable the titlebar (hamburgermenu > customize toolbar > check titlebar)
1. Install a lightweight theme, or the default Light or Dark themes
Actual results:
The titlebar keeps the Mac system color, the lightweight theme is only visible in the tab, tool and bookmarks bars.
Expected results:
The theme background should be visible in the titlebar as well as in tabs, toolbar, bookmarks bar
Additional info:
- This didn’t happen in Firefox 54. See the attached image for comparisons.
- I use Mac OS X Yosemite 10.10.5
Updated•7 years ago
|
Component: Untriaged → Theme
OS: Unspecified → Mac OS X
Updated•7 years ago
|
Blocks: 1392219
Status: UNCONFIRMED → NEW
status-firefox57:
--- → wontfix
status-firefox58:
--- → wontfix
status-firefox59:
--- → affected
Ever confirmed: true
Keywords: regression
Priority: -- → P5
Comment 3•7 years ago
|
||
(In reply to Oana Botisan from comment #2)
> I can reproduce this bug on Windows 10 x64 as well.
You're talking about bug 1381004. I can understand that this would seem like the same issue to you, but the underlying issues are vastly different between the two platforms, so it doesn't really make sense to track them in a single bug. :)
Flags: qe-verify+
OS: All → Mac OS X
Hardware: Unspecified → All
See Also: → 1381004
Summary: Titlebar renders differently in default and lightweight themes. → MacOS titlebar renders differently in default and lightweight themes.
Comment 5•7 years ago
|
||
To come back to this, I believe the original issue was that we weren't able to change the dark titlebar text to be lighter when dealing with e.g. the default theme, which has a dark titlebar. I don't remember whether we fully exhausted our options here or if this was just dropped in favor of more important Photon work.
Markus, on the off chance, is there a way we can change the native OSX titlebar text color, save from drawing the entire titlbar ourselves (bug 1381004)?
Thanks!
Flags: needinfo?(mstange)
Comment 6•7 years ago
|
||
If it's not possible to draw the titlebar text lighter, what about having the theme draw in the titlebar as long as it's not a dark theme?
In Thunderbird 57 we had Photon tabs but themes still drew in the titlebar, which looked quite nice, see the attachment. The new version in 58/59 is a bit ugly in comparison.
Most people have the titlebar on in Thunderbird not because they want to see the titlebar text, but just because they choose mail.tabs.autohide, to hide the tab when there's only one, which forces the titlebar on. Also the default theme isn't dark.
Comment 7•7 years ago
|
||
(In reply to Johann Hofmann [:johannh] from comment #5)
> Markus, on the off chance, is there a way we can change the native OSX
> titlebar text color, save from drawing the entire titlbar ourselves (bug
> 1381004)?
Only with extreme contortions in the widget code, which I'd really prefer to avoid.
Flags: needinfo?(mstange)
Comment 8•7 years ago
|
||
Maybe a hidden preference to get the old behaviour back then? Most themes work fine with it, and it looks so much better.
Comment 9•7 years ago
|
||
(In reply to Elhem Enohpi from comment #8)
> Maybe a hidden preference to get the old behaviour back then? Most themes
> work fine with it, and it looks so much better.
Yes, I think we'd take a patch for this, but I doubt anyone from the Firefox team is going to work on it for now.
Updated•7 years ago
|
status-firefox60:
--- → wontfix
status-firefox61:
--- → fix-optional
Comment 10•7 years ago
|
||
We've decided this is the way we want the titlebar to look. Hence, invalid.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•