From df913bdc50b202d9a13a2304297a370ebb1ceffe Mon Sep 17 00:00:00 2001 From: Vlad Zahorodnii Date: Fri, 14 Jan 2022 15:20:24 +0200 Subject: [PATCH] Remove some unintuitive code in AbstractClient::setQuickTileMode() After calling setMaximize(), the window should cover the maximize area. The commit message of 516ea863419084010eac1310582a08a46bfa95ad doesn't explain why only the y coordinate is enforced or why the x coordinate is not enforced. In order to make some sense of the code, this change removes the semi- random enforcement of the y coordinate. --- src/abstract_client.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/abstract_client.cpp b/src/abstract_client.cpp index 1edd934d36..9faa4ff00f 100644 --- a/src/abstract_client.cpp +++ b/src/abstract_client.cpp @@ -3087,12 +3087,6 @@ void AbstractClient::setQuickTileMode(QuickTileMode mode, bool keyboard) QRect prev_geom_restore = geometryRestore(); // setMaximize() would set moveResizeGeom as geom_restore m_quickTileMode = int(QuickTileFlag::Maximize); setMaximize(true, true); - QRect clientArea = workspace()->clientArea(MaximizeArea, this); - if (moveResizeGeometry().top() != clientArea.top()) { - QRect r(moveResizeGeometry()); - r.moveTop(clientArea.top()); - moveResize(r); - } setGeometryRestore(prev_geom_restore); } doSetQuickTileMode();