! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Undecorate on maximize
Status:
RESOLVED: FIXED
Severity:
enhancement

Comments

Description Viktor Semykin 2013-03-10 23:29:01 CET
Add the ability to undecorate maximized windows to save vertical space on screen.
Comment 1 Viktor Semykin 2013-03-10 23:29:20 CET
Implemented here https://bitbucket.org/thesame/xfwm4-titleless
Comment 2 Olivier Fourdan editbugs 2013-03-11 21:09:05 CET
(In reply to comment #1)
> Implemented here https://bitbucket.org/thesame/xfwm4-titleless

Please attach patches here if you want this upstream (can't really teach changes from another git repo,not convenient...)
Comment 3 Olivier Fourdan editbugs 2013-03-11 21:19:46 CET
Note, title-less maximization is already covered by EWMH, so this needs to be implemented via the standard, making it an unconditional option in not the right way to add this feature.

1. add support for EWMH standard for borderless maximization
2. make sure it's not just the title, but border as we'll (ie there is already an option to show or hide borders on maximization, make sure this work in any case)
3. once the feature supports the standard, we can add a user option to set the default behaviour, for apps that don't specify their desired maximization mode via the standard.
Comment 4 Viktor Semykin 2013-03-12 02:11:25 CET
Created attachment 4959 
Adds ability to hide titles of maximized windows
Comment 5 Viktor Semykin 2013-03-12 02:11:50 CET
(In reply to comment #3)
> Note, title-less maximization is already covered by EWMH, so this needs to
> be implemented via the standard, making it an unconditional option in not
> the right way to add this feature.

I couldn't find appropriate EWMH atom to show/hide titles of maximized windows. My solution isn't unconditional, it adds new checkbox (initially unchecked) to WM tweaks.

> 1. add support for EWMH standard for borderless maximization
> 2. make sure it's not just the title, but border as we'll (ie there is
> already an option to show or hide borders on maximization, make sure this
> work in any case)

My solution covers both full and vertical titleless maximizations. I'd be glad if someone tested it with me. It works for me and I already use it.
Comment 6 koct9i 2013-06-10 11:28:09 CEST
I have applied this to xfwm4 4.10.1 from debian/testing and it work like a charm.
Patch little bit messy, it would be better to split it into pieces.

screenshot: https://plus.google.com/105141859736228162791/posts/QpiEk7uf6Ah
Comment 7 Olivier Fourdan editbugs 2013-06-10 16:26:35 CEST
(In reply to comment #5)
> 
> I couldn't find appropriate EWMH atom to show/hide titles of maximized
> windows. My solution isn't unconditional, it adds new checkbox (initially
> unchecked) to WM tweaks.

True, indeed, I got confused.

But gtk+ has a mechanism for that: "_GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED"

https://git.gnome.org/browse/gtk+/tree/gdk/x11/gdkwindow-x11.c#n3472

Would be great to have that as well, so that apps that use this get undecorated when maximized even if the user has not specified the option (so we geta consistent behavior with gtk+ apps at least)
Comment 8 Viktor Semykin 2013-07-12 07:19:53 CEST
Created attachment 5092 
_GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED support

Diff against current git master. Must be applied on top of previous patch.
Comment 9 Viktor Semykin 2013-08-23 01:21:10 CEST
Nobody seems to care :(
Comment 10 Christian Hesse 2013-09-26 11:16:18 CEST
I do care and I really like it! ;)
This should be merged upstream I think.
Comment 11 Cédric Leporcq 2013-11-29 18:28:27 CET
Hello, I've worked on this patch made by Viktor Semykin and I've made some improvements and fixes for xfwm4 4.10 and 4.11.
If  you are interested it's available here: https://github.com/cedl38/xfwm4-titleless.
Comment 12 Cédric Leporcq 2013-11-29 19:07:32 CET
Created attachment 5245 
add xfwm4 titleless option for maximized windows - 4.10.1
Comment 13 Cédric Leporcq 2013-11-29 19:10:13 CET
Comment on attachment 5245 
add xfwm4 titleless option for maximized windows - 4.10.1

patch for xfwm4 version 4.10.1
Comment 14 Cédric Leporcq 2013-11-29 19:19:17 CET
Created attachment 5246 
hide title and top border on maximized features - 4.10.1

Patch for xfwm4 version 4.10.1
Add titleless maximization feature and extra functionalities: 
- Change top border creation for hiding capabilities (add frameTopBorder and frameTitle functions)
- Split title-*.xpm into title and top border part if top-3-active.xpm don't exist
- add optional top_border_height and top_border_maximize parameters.
Comment 15 Cédric Leporcq 2013-11-29 19:26:31 CET
Created attachment 5247 
hide title and top border on maximized features - 4.11.0

Patch for xfwm4 version 4.11.0
Add titleless maximization feature and extra functionalities.
Comment 16 Christian Hesse 2014-01-03 09:57:54 CET
Created attachment 5292 
Screenshot of misrendered decorations

Using the latest patch by Cédric Leporcq now. Works great on most system, though it interferes with bug 10049 [0]. Uploading a screenshot of misrendered decorations, please compare with the upload from the other bug. Any ideas what could have caused this?

https://bugzilla.xfce.org/show_bug.cgi?id=10049
Comment 17 Cédric Leporcq 2014-01-03 14:24:54 CET
Hello Christian,

Can you please tell me witch theme are you using ? I’m wondering if this bug can be related to a specific theme.

My latest patch have made some changes regarding the theme implementation to allow top border hiding. Theses changes d’ont affect theme rendering for most of them. I noticed some glitch on few themes (like metabox) for very particular reason that can be easily fixed with theme updates.

(my latest revisions can be found here : https://github.com/cedl38/xfwm4-titleless)
Comment 18 Christian Hesse 2014-01-03 16:03:55 CET
Hello Cédric,

this is a customized version of Curve, but plain Curve from xfwm4-themes suffers the same issue. Looks like this happens whenever just one visible pixel in a row...

I do see this in a virtualbox with guest additions installed. Bare metal with intel hardware in uneffected, so not sure if the is a xfwm4 or driver problem.

What did you change for Metabox? Perhaps that gives me any idea how to fix it.
Comment 19 Christian Hesse 2014-01-03 16:29:20 CET
Probably worst one is Cruxish... It misses the title entirly. title-3-(in)?active.xpm is just one pixel wide, so it fits my theory.
Comment 20 Cédric Leporcq 2014-01-03 18:24:01 CET
Created attachment 5296 
metabox bug

I can’t reproduce yours bugs on my system. To test your theory you can change the pixmaps wide from 1 pixel to 3 for example with a text editor. This change does not affect the theme appearance.

For metabox there is a bug in the theme, but only visible with my patch.
To fix it change number 2 by 3 in title_5_active_xpm with a text editor:

"8 21 2 1",
"     c None",
".    c #000000",
"+    c #5B80AD s active_color_1",

"8 21 2 1", should be "8 21 3 1", because there are three colors in this pixmap
Comment 21 Christian Hesse 2014-01-05 15:18:53 CET
I think my xpm images are ok.

Looks like this only happens when X.org VESA driver is in action (which is also true for virtualbox guests whith vboxvideo). So possibly we do not have a xfwm4 bug but a X.org driver one.
Comment 22 Landry Breuil editbugs 2014-11-21 13:58:07 CET
*** Bug 7436 has been marked as a duplicate of this bug. ***
Comment 23 Olivier Fourdan editbugs 2015-01-11 18:08:36 CET
Implemented with commit 2c77f62

Bug #9905

Reported by:
Viktor Semykin
Reported on: 2013-03-10
Last modified on: 2015-01-11
Duplicates (1):
  • 7436 Window top border is still present when maximized

People

Assignee:
Olivier Fourdan
CC List:
7 users

Version

Version:
unspecified

Attachments

Additional information