! 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 !
Window top border is still present when maximized


Description Mark 2011-03-23 12:42:09 CET

When a window is maximized so it fills the whole screen, its top border is still shown. The top border in this state is useless though, since it cannot be dragged to resize the maximized window. So it just wastes space.

For most xfwm4 styles/themes this doesn't actually look too bad. But with a theme with relatively thick window borders it can look quite ugly. To see what I mean, change the window manager style to "Mofit" then maximize a window.
Comment 1 Mark 2011-12-15 21:28:42 CET
I might have found the cause of the problem.

In src/frame.c, the frameTop() definition does not have similar checks for CLIENT_FLAG_MAXIMIZED as the frameLeft(), frameRight() and frameBottom() functions.

Should the frameTop() function be changed to read something like this?

frameTop (Client * c)
    TRACE ("entering frameTop");

    g_return_val_if_fail (c != NULL, 0);
    if (FLAG_TEST (c->xfwm_flags, XFWM_FLAG_HAS_BORDER)
        && !FLAG_TEST (c->flags, CLIENT_FLAG_FULLSCREEN)
        && (!FLAG_TEST_ALL (c->flags, CLIENT_FLAG_MAXIMIZED)     // This condition added
            || !(c->screen_info->params->borderless_maximize)))  // This condition added
        return c->screen_info->title[TITLE_3][ACTIVE].height;
    return 0;
Comment 2 Olivier Fourdan editbugs 2011-12-15 21:56:53 CET
Removing the top border when maximized would break most themes, not something I would like just for a few themes such as the Motif like theme.

Not something I would fix.
Comment 3 Cédric Leporcq 2013-12-08 22:04:17 CET

I’ve managed to implement this feature without breaking old themes.

look at #9905
Comment 4 Landry Breuil editbugs 2014-11-21 13:58:07 CET
See #9905 for work on this and patches, resolving this one as duplicate

*** This bug has been marked as a duplicate of bug 9905 ***

Bug #7436

Reported by:
Reported on: 2011-03-23
Last modified on: 2014-11-21


Olivier Fourdan
CC List:
2 users



Additional information