davidedmundson added inline comments.

INLINE COMMENTS

> davidedmundson wrote in xinputintegration.cpp:132
> I think we want to always update m_lastTouchPositions outside of 
> if (e->detail == m_trackingTouchId)
> 
> otherwise if we get:
> 
> TouchBegin
> TouchUpdate
> TouchOwnership
> 
> we start the gesture in the wrong position

Sorry that's not quite what i meant

m_lastTouchPositions now needs to use e->detail

And the gestureRecognizer stuff should still be inside the if.

REVISION DETAIL
  https://phabricator.kde.org/D5137

To: graesslin, #plasma, #kwin
Cc: davidedmundson, plasma-devel, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol

Reply via email to