From 97dbbfbe2c7a1609119263bf09eded9c5bf2801a Mon Sep 17 00:00:00 2001 From: Vlad Zagorodniy Date: Mon, 22 Jul 2019 23:28:28 +0300 Subject: [PATCH] Pass arguments to changeMaximize in correct order Summary: The first argument in AbstractClient::changeMaximize specifies whether the maximized state of the client needs to be toggled in horizontal direction. Reviewers: #kwin, romangg Reviewed By: #kwin, romangg Subscribers: kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D22486 --- geometry.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/geometry.cpp b/geometry.cpp index 3860274e4f..27ec3cd2cc 100644 --- a/geometry.cpp +++ b/geometry.cpp @@ -2148,8 +2148,8 @@ void AbstractClient::setMaximize(bool vertically, bool horizontally) // changeMaximize() flips the state, so change from set->flip const MaximizeMode oldMode = maximizeMode(); changeMaximize( - oldMode & MaximizeVertical ? !vertically : vertically, oldMode & MaximizeHorizontal ? !horizontally : horizontally, + oldMode & MaximizeVertical ? !vertically : vertically, false); const MaximizeMode newMode = maximizeMode(); if (oldMode != newMode) { @@ -2188,7 +2188,7 @@ private: static bool changeMaximizeRecursion = false; -void Client::changeMaximize(bool vertical, bool horizontal, bool adjust) +void Client::changeMaximize(bool horizontal, bool vertical, bool adjust) { if (changeMaximizeRecursion) return;