[Linux] Polish titlebar config
Categories
(Core :: Widget: Gtk, task)
Tracking
()
People
(Reporter: stransky, Assigned: stransky)
References
(Blocks 1 open bug)
Details
Attachments
(3 files, 2 obsolete files)
Right now we don't test titlebar rendering in mozilla testsuite. That leads to various issues and regressions.
Assignee | ||
Comment 1•1 year ago
|
||
Assignee | ||
Comment 2•1 year ago
|
||
Recently -moz-gtk-csd-hide-titlebar-by-default is used to control initial state of titlebar rendering on Linux.
This is used on Linux only, such preference confuses Mozilla test suite which expects browser.tabs.drawInTitlebar only
as it's used on other platforms.
Let's align Linux behaviors with Win/Mac and set browser.tabs.drawInTitlebar directly at browser start (if it's not already set).
Depends on D128995
Comment 3•1 year ago
|
||
To more properly support Linux having a different default at runtime.
Expose the resolved value in appinfo for convenience, and use it in the
front-end as needed.
Comment 4•1 year ago
|
||
This is a regression from bug 1681356. nsWindow::GtkWindowDecoration()
default-initializes a GtkWindowDecoration, which ends up being
GTK_DECORATION_SYSTEM since that's the first variant.
Fix the check by using the proper function
GetSystemGtkWindowDecoration(). Quite a footgun...
Depends on D129004
Comment 5•1 year ago
|
||
Clean up the check while at it.
Depends on D129005
Updated•1 year ago
|
Updated•1 year ago
|
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/62d204b08dce Make browser.tabs.drawInTitlebar a tri-state. r=stransky,Gijs https://hg.mozilla.org/integration/autoland/rev/ad3d2766d9da Fix CSD check in nsLookAndFeel. r=stransky https://hg.mozilla.org/integration/autoland/rev/537090f8fa57 Remove GTK_CSD_DECORATION_UNKNOWN. r=stransky
Assignee | ||
Updated•1 year ago
|
Comment 7•1 year ago
|
||
bugherder |
Comment 8•1 year ago
|
||
Should this bug still be open? Maybe it would better to create a new followup bug if there is going to be more later.
Comment 9•1 year ago
|
||
Yeah maybe we should retitle this bug and close it? WDYT Martin?
Assignee | ||
Comment 10•1 year ago
|
||
There's try with enabled CSD in try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=018a3bcf76022f97eb1481d95c4d83aebed002e8
so let's see how it works.
Assignee | ||
Comment 11•1 year ago
|
||
Hm, the try does not look good. Okay, let's close it and file another one for CSD enabled in testsuite.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Description
•