Bug 54730 - window(decoration) border cut off when maximizing
Summary: window(decoration) border cut off when maximizing
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Mandrake RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-16 19:50 UTC by Yves Glodt
Modified: 2003-03-31 09:45 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yves Glodt 2003-02-16 19:50:16 UTC
Version:            (using KDE KDE 3.1)
Installed from:    Mandrake RPMs

Hi,

when you deselect "Allow moving and resizing of maximized windows", the window decoration suffers, because the vertical resize-border is being removed.
This unfortunately cuts off the titlebar aswell and
thus makes the window look incomplete, which is ugly :-(

This is not a new issue but exists for a long time.

The resize-border should not go away, but simply not react to the 
mouse anymore.
Comment 1 Lubos Lunak 2003-03-19 16:42:42 UTC
Subject: kwin_iii: kdebase/kwin

CVS commit by lunakl: 

Don't try to "remove" resize borders when the window is maximalized
and moving of maximalized windows is disabled.
This is just causing trouble now, most decorations can't handle it.
Moreover it can't work well with size increments.
If ever this feature will be enabled again, it needs to be done in styles.

CCMAIL: 54730-done@bugs.kde.org


  M +0 -8      client.cpp   1.344.2.25


--- kdebase/kwin/client.cpp  #1.344.2.24:1.344.2.25
@@ -2432,12 +2432,4 @@ void Client::changeMaximize( bool vertic
         QSize adjSize = adjustedSize(clientArea.size());
         QRect r = QRect(clientArea.topLeft(), adjSize);
-
-        // hide right and left border of maximized windows
-        if ( !options->moveResizeMaximizedWindows && adjSize == clientArea.size()) {
-            if ( r.left() == 0 )
-                r.setLeft( r.left() - windowWrapper()->x() );
-            if ( r.right() == workspace()->geometry().right() )
-                r.setRight( r.right() + width() -  windowWrapper()->geometry().right() );
-        }
         setGeometry( r );
         info->setState( NET::Max, NET::Max );


Comment 2 John Firebaugh 2003-03-31 09:45:50 UTC
Unfortunately, this is a highly desireable feature from a UI standpoint, and I wish it 
hadn't been removed. The reason is Fitts's Law. With the window border of a maximized 
window hidden, any UI elements on the inside border, e.g. scrollbars or tab widgets, are 
directly on the screen edge and thus have 'infinite' depth, making them easy targets to 
hit with the mouse.