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

Fix registering touch screen edge actions after the screen edge has been reserved

Browse source 
After the screen edge is reserved, the touch screen gestures will be
registered or unregistered when the Edge::activatesForTouchGesture()
signal is emitted.

On the other hand, Edge::reserveTouch() lacks code to emit that signal,
which results in touch screen gesture not working if the same screen
edge is reserved both for pointer and touch input.

BUG: 451349
This commit is contained in:
Vlad Zahorodnii 2024-06-12 14:32:06 +03:00
parent 35a7e30952
commit 6be238e622
1 changed files with 13 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/screenedge.cpp
View file

@ -188,10 +188,14 @@ void Edge::reserveTouchCallBack(const TouchCallback &callback)
})) {
return;
}
const bool wasTouch = activatesForTouchGesture();
connect(callback.touchUpAction(), &QAction::destroyed, this, [this, callback]() {
unreserveTouchCallBack(callback.touchUpAction());
});
m_touchCallbacks << callback;
if (wasTouch != activatesForTouchGesture()) {
Q_EMIT activatesForTouchGestureChanged();
}
reserve();
}
@ -200,10 +204,16 @@ void Edge::unreserveTouchCallBack(QAction *action)
auto it = std::find_if(m_touchCallbacks.begin(), m_touchCallbacks.end(), [action](const TouchCallback &c) {
return c.touchUpAction() == action;
});
if (it != m_touchCallbacks.end()) {
m_touchCallbacks.erase(it);
unreserve();
if (it == m_touchCallbacks.end()) {
return;
}
const bool wasTouch = activatesForTouchGesture();
m_touchCallbacks.erase(it);
if (wasTouch != activatesForTouchGesture()) {
Q_EMIT activatesForTouchGestureChanged();
}
unreserve();
}
void Edge::unreserve()