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

Use flooredPoint() in pointer constraint tests

Browse source 
.toPoint() doesn't work as expected when the pointer has fractional
coordinates and it's near constraint region boundaries.
This commit is contained in:
Vlad Zahorodnii 2023-05-12 12:49:15 +03:00
parent 6c71b87200
commit 5ccb543e89
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/pointer_input.cpp
View file

@ -632,7 +632,7 @@ void PointerInputRedirection::updatePointerConstraints()
}
return;
}
if (canConstrain && cf->region().contains(focus()->mapToLocal(m_pos).toPoint())) {
if (canConstrain && cf->region().contains(flooredPoint(focus()->mapToLocal(m_pos)))) {
cf->setConfined(true);
m_confined = true;
m_confinedPointerRegionConnection = connect(cf, &KWaylandServer::ConfinedPointerV1Interface::regionChanged, this, [this]() {
@ -644,7 +644,7 @@ void PointerInputRedirection::updatePointerConstraints()
return;
}
const auto cf = s->confinedPointer();
if (!cf->region().contains(focus()->mapToLocal(m_pos).toPoint())) {
if (!cf->region().contains(flooredPoint(focus()->mapToLocal(m_pos)))) {
// pointer no longer in confined region, break the confinement
cf->setConfined(false);
m_confined = false;
@ -675,7 +675,7 @@ void PointerInputRedirection::updatePointerConstraints()
}
return;
}
if (canConstrain && lock->region().contains(focus()->mapToLocal(m_pos).toPoint())) {
if (canConstrain && lock->region().contains(flooredPoint(focus()->mapToLocal(m_pos)))) {
lock->setLocked(true);
m_locked = true;