Commit df913bdc authored by Vlad Zahorodnii's avatar Vlad Zahorodnii
Browse files

Remove some unintuitive code in AbstractClient::setQuickTileMode()

After calling setMaximize(), the window should cover the maximize area.
The commit message of 516ea863 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.
parent 3ba316e6
Pipeline #124660 passed with stage
in 15 minutes and 38 seconds
......@@ -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();
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment