From e1ede03c87c3fd1c30869fdf6e384327d0a79c8f Mon Sep 17 00:00:00 2001 From: Michail Vourlakos Date: Thu, 19 Mar 2020 12:49:39 +0200 Subject: [PATCH] drop QRect::setX/Y from geometry calculations --these functions can change also the QRect width and height. moveLeft/Top are the functions that provided the best behavior. --- app/view/screenedgeghostwindow.cpp | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/view/screenedgeghostwindow.cpp b/app/view/screenedgeghostwindow.cpp index 0be175df8..7ebe50e03 100644 --- a/app/view/screenedgeghostwindow.cpp +++ b/app/view/screenedgeghostwindow.cpp @@ -210,20 +210,20 @@ void ScreenEdgeGhostWindow::updateGeometry() if (m_latteView->location() == Plasma::Types::BottomEdge) { int xF = qMax(m_latteView->screenGeometry().left(), m_latteView->absoluteGeometry().left() - lengthDifference); - newGeometry.setX(xF); - newGeometry.setY(m_latteView->screenGeometry().bottom() - thickness); + newGeometry.moveLeft(xF); + newGeometry.moveTop(m_latteView->screenGeometry().bottom() - thickness); } else if (m_latteView->location() == Plasma::Types::TopEdge) { int xF = qMax(m_latteView->screenGeometry().left(), m_latteView->absoluteGeometry().left() - lengthDifference); - newGeometry.setX(xF); - newGeometry.setY(m_latteView->screenGeometry().top()); + newGeometry.moveLeft(xF); + newGeometry.moveTop(m_latteView->screenGeometry().top()); } else if (m_latteView->location() == Plasma::Types::LeftEdge) { int yF = qMax(m_latteView->screenGeometry().top(), m_latteView->absoluteGeometry().top() - lengthDifference); - newGeometry.setX(m_latteView->screenGeometry().left()); - newGeometry.setY(yF); + newGeometry.moveLeft(m_latteView->screenGeometry().left()); + newGeometry.moveTop(yF); } else if (m_latteView->location() == Plasma::Types::RightEdge) { int yF = qMax(m_latteView->screenGeometry().top(), m_latteView->absoluteGeometry().top() - lengthDifference); - newGeometry.setX(m_latteView->screenGeometry().right() - thickness); - newGeometry.setY(yF); + newGeometry.moveLeft(m_latteView->screenGeometry().right() - thickness); + newGeometry.moveTop(yF); } if (m_latteView->formFactor() == Plasma::Types::Horizontal) {