[GTK+ 3.10.8 / JWM] Titlebar drawing not eneabled when XDG_CURRENT_DESKTOP is missing
Categories
(Core :: Widget: Gtk, defect, P5)
Tracking
()
People
(Reporter: terrell.kelley, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(5 files, 1 obsolete file)
This was originally implemented in Bug 1414212 but was temporarily removed in order to fix a regression (Bug 1423810). The regression was fixed in Bug 1424974, so let's reenable for GTK+ >= 3.10 (Note: This was already reenabled for GTK 3.20+ in Bug 1417933.)
Reporter | ||
Updated•6 years ago
|
Reporter | ||
Updated•6 years ago
|
Reporter | ||
Comment 1•6 years ago
|
||
Note: I have made the assumption that this was just not reenabled for GTK+ >= 3.10. It could possibly be something else. SPECS Ubuntu 14.04 LTS GTK+ version 3.10.8] JWM Window Manager (https://joewing.net/projects/jwm/)
Comment 2•6 years ago
|
||
Can you please run latest trunk and check what settings do work for you? Set browser.tabs.drawInTitlebar to true at about:config and run: $MOZ_GTK_TITLEBAR_DECORATION=client firefox $MOZ_GTK_TITLEBAR_DECORATION=system firefox the "client" should work at least.
Updated•6 years ago
|
Reporter | ||
Comment 3•6 years ago
|
||
I will assume "latest trunk" means the latest Nightly. Both of those work (without the $). But ./firefox by itself does not.
Comment 4•6 years ago
|
||
Thanks, that looks good. Can you please also attach a screenshot of firefox at client and system mode running on some white background? I'd like to see how the window borders and resize grips are rendered and if there are shadows round the main window. Also please attach a content of XDG_CURRENT_DESKTOP env variable, you can get it by: env | grep XDG_CURRENT_DESKTOP on terminal. Thanks.
Reporter | ||
Comment 5•6 years ago
|
||
Reporter | ||
Comment 6•6 years ago
|
||
Reporter | ||
Comment 7•6 years ago
|
||
I have also included a version without either set, in case that is relevant for your second paragraph. As you can see, there are no shadows, and no indicators for the resizers (the cursor just changes). Unfortunately, it seems that JWM does not currently set $XDG_CURRENT_DESKTOP, nor any similar variables. I do note that `wmctrl -m` correctly detects the window manager as JWM. I also note that my previous bug 1424158 is still present, which is why I have a broken image on the close button, and no other window control buttons. I am using the Compact density setting. Finally, I note that the background is slightly off white, as I used the new tab page as the background. I didn't notice until after I had already uploaded the first two images.
Reporter | ||
Comment 8•6 years ago
|
||
Quick additional question: Would getting the borders help with fixing the fact that the border appears on top, even when maximized? Should I maybe also attach an image of what it looks like when maximized? Also, note that I included that about `wmctrl -m` to say that at least some apps can detect the WM. I'm hoping this one can be detected, too. Though, worst case, I'll just modify my start script to set that variable to JWM before launching Firefox.
Comment 9•6 years ago
|
||
(In reply to Terrell Kelley from comment #7) > Created attachment 8944065 [details] > no options set > > I have also included a version without either set, in case that is relevant > for your second paragraph. As you can see, there are no shadows, and no > indicators for the resizers (the cursor just changes). > > Unfortunately, it seems that JWM does not currently set > $XDG_CURRENT_DESKTOP, nor any similar variables. I do note that `wmctrl -m` > correctly detects the window manager as JWM. > > I also note that my previous bug 1424158 is still present, which is why I > have a broken image on the close button, and no other window control > buttons. I am using the Compact density setting. > > Finally, I note that the background is slightly off white, as I used the new > tab page as the background. I didn't notice until after I had already > uploaded the first two images. This is a bug at JWM - it should set XDG_CURRENT_DESKTOP shomehow. We don't enable titlebar drawing when XDG_CURRENT_DESKTOP is not set right now.
Updated•6 years ago
|
Comment 10•5 years ago
|
||
Updated•2 years ago
|
Comment 12•15 days ago
|
||
This has never been fixed.
Assignee | ||
Comment 13•14 days ago
|
||
Updated•14 days ago
|
Comment 15•12 days ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b39ab94454e9 Allow CSD even without XDG_CURRENT_DESKTOP. r=stransky
Comment 16•11 days ago
•
|
||
Backed out changeset b39ab94454e9 (bug 1431429) for causing mochitest failures @ test_arrowpanel.html / browser_unified_extensions_overflowable_toolbar.js & dt failures @ browser_html_tooltip_screen_edge.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/1b56c653a5ee3a14ec0955b092498999e9333bfa
Assignee | ||
Comment 17•11 days ago
|
||
This fixes the DevTools time-out that got me backed out below. In Linux
with CSD enabled, the allowable screen range for a window should include
some extra size (to account for the window decorations).
See some of the other users of these properties in the tree.
Before this bug, the relevant code is not exercised on automation.
Assignee | ||
Comment 18•11 days ago
|
||
Blegh, so my try run used artifact builds so that's why it didn't catch comment 16, fail on my side.
Martin, how worth it do you think this is? Comment 17 shows some of the kinds of fixes that are needed, but a lot of the existing failures aren't reproducible on my end (even on GNOME+X11), so it's probably not-low-effort to fix... If you think it's worth it I can take fixing all this stuff up over time, I guess...
Comment 19•11 days ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #18)
Blegh, so my try run used artifact builds so that's why it didn't catch comment 16, fail on my side.
Martin, how worth it do you think this is? Comment 17 shows some of the kinds of fixes that are needed, but a lot of the existing failures aren't reproducible on my end (even on GNOME+X11), so it's probably not-low-effort to fix... If you think it's worth it I can take fixing all this stuff up over time, I guess...
I think we can leave it after testsuite migration to 22.04 and set XDG_CURRENT_DESKTOP there so we test what we ship. I don't think it's worth to spend time on it right now.
Comment 20•9 days ago
|
||
Updated•9 days ago
|
Comment 22•8 days ago
|
||
Yes sorry, I was experimenting with something, botched a rebase, and accidentally pushed to phabricator. I've abandoned https://phabricator.services.mozilla.com/D207175 ...
Description
•