yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ensure move() calls in placement move in integer values

Browse source 
Otherwise we run the risk of placing windows at floating coodinates,
which leads to interpolated rendering which doesn't look nice.

BUG: 457143
This commit is contained in:
Arjen Hiemstra 2022-09-14 10:23:06 +02:00 committed by Vlad Zahorodnii
parent b001741634
commit 817d13d8c0
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/placement.cpp
View file

@ -539,7 +539,7 @@ void Placement::placeUnderMouse(Window *c, const QRect &area, PlacementPolicy /*
QRectF geom = c->frameGeometry(); QRectF geom = c->frameGeometry();
geom.moveCenter(Cursors::self()->mouse()->pos()); geom.moveCenter(Cursors::self()->mouse()->pos());
c->move(geom.topLeft()); c->move(geom.topLeft().toPoint());
c->keepInArea(area); // make sure it's kept inside workarea c->keepInArea(area); // make sure it's kept inside workarea
} }
@ -668,7 +668,7 @@ void Window::packTo(qreal left, qreal top)
workspace()->updateFocusMousePosition(Cursors::self()->mouse()->pos()); // may cause leave event; workspace()->updateFocusMousePosition(Cursors::self()->mouse()->pos()); // may cause leave event;
const Output *oldOutput = moveResizeOutput(); const Output *oldOutput = moveResizeOutput();
move(QPointF(left, top)); move(QPoint(left, top));
if (moveResizeOutput() != oldOutput) { if (moveResizeOutput() != oldOutput) {
workspace()->sendWindowToOutput(this, moveResizeOutput()); // checks rule validity workspace()->sendWindowToOutput(this, moveResizeOutput()); // checks rule validity
if (requestedMaximizeMode() != MaximizeRestore) { if (requestedMaximizeMode() != MaximizeRestore) {