Skip to content

Commit

Permalink
Merge pull request #80 from adenexter/jb30851
Browse files Browse the repository at this point in the history
Fixes for mishandled touch release and cancel events.
  • Loading branch information
sletta committed Sep 15, 2015
2 parents 016381e + 362b450 commit 277b8cf
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/quick/items/qquickflickable.cpp
Expand Up @@ -1251,7 +1251,7 @@ void QQuickFlickable::mouseReleaseEvent(QMouseEvent *event)
if (window() && window()->mouseGrabberItem()) {
QPointF localPos = window()->mouseGrabberItem()->mapFromScene(event->windowPos());
QScopedPointer<QMouseEvent> mouseEvent(QQuickWindowPrivate::cloneMouseEvent(event, &localPos));
window()->sendEvent(window()->mouseGrabberItem(), mouseEvent.data());
QCoreApplication::sendEvent(window(), mouseEvent.data());
}

// And the event has been consumed
Expand Down Expand Up @@ -2059,6 +2059,7 @@ bool QQuickFlickable::sendMouseEvent(QQuickItem *item, QMouseEvent *event)
break;
case QEvent::MouseButtonRelease:
d->handleMouseReleaseEvent(mouseEvent.data());
stealThisEvent = d->stealMouse;
break;
default:
break;
Expand Down
1 change: 1 addition & 0 deletions src/quick/items/qquickmousearea.cpp
Expand Up @@ -867,6 +867,7 @@ bool QQuickMouseArea::sendMouseEvent(QMouseEvent *event)
break;
case QEvent::MouseButtonRelease:
mouseReleaseEvent(&mouseEvent);
stealThisEvent = d->stealMouse;
break;
default:
break;
Expand Down
6 changes: 6 additions & 0 deletions src/quick/items/qquickmultipointtoucharea.cpp
Expand Up @@ -412,6 +412,12 @@ void QQuickMultiPointTouchArea::touchEvent(QTouchEvent *event)
}
break;
}
case QEvent::TouchCancel:
_stealMouse = false;
setKeepMouseGrab(false);
setKeepTouchGrab(false);
ungrabTouchPoints();
break;
default:
QQuickItem::touchEvent(event);
break;
Expand Down

0 comments on commit 277b8cf

Please sign in to comment.