From cfa5daaad1bfb954e7b2176bba7b985410584e63 Mon Sep 17 00:00:00 2001 From: Vlad Zahorodnii Date: Mon, 17 Feb 2020 14:10:46 +0200 Subject: [PATCH] Fix misplaced client windows Summary: This regression was introduced by me, sorry. The client window may be at location other than (0, 0), in which case we have to move it to (0, 0) to ensure that the client contents is not covered by window frame. BUG: 417584 FIXED-IN: 5.18.1 Reviewers: #kwin, broulik Reviewed By: broulik Subscribers: broulik, kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D27456 --- x11client.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/x11client.cpp b/x11client.cpp index 15b8a6fdeb..ba7f103de0 100644 --- a/x11client.cpp +++ b/x11client.cpp @@ -4303,7 +4303,7 @@ void X11Client::updateServerGeometry() if (!isShade()) { if (needsGeometryUpdate) { m_wrapper.setGeometry(QRect(clientPos(), clientSize())); - m_client.resize(clientSize()); + m_client.setGeometry(QRect(QPoint(0, 0), clientSize())); } // SELI - won't this be too expensive? // THOMAS - yes, but gtk+ clients will not resize without ... @@ -4792,7 +4792,7 @@ void X11Client::doResizeSync() // this, then we might render partially updated client window. I know, it sucks. m_frame.setGeometry(moveResizeBufferGeometry); m_wrapper.setGeometry(QRect(clientPos(), moveResizeClientGeometry.size())); - m_client.resize(moveResizeClientGeometry.size()); + m_client.setGeometry(QRect(QPoint(0, 0), moveResizeClientGeometry.size())); } void X11Client::doPerformMoveResize()