on xenial help sign in
Make sure dnd mode is ended properly when drag gesture is cancelled
commitb27711268cf26044d02124999351e337899bd2ab authorMichael Zanetti <michael.zanetti@canonical.com> dateThu 16 Apr 2015, 12:52 (12 years ago) parent7478eb8
2 files changed+35
qml/Launcher/LauncherPanel.qml+4 tests/qmltests/Launcher/tst_Launcher.qml+31
qml/Launcher/LauncherPanel.qml+4
@@ -338,7 +338,11 @@ Rectangle {
338338 }
339339
340340 onCanceled: {
341+ if (selectedItem) {
342+ selectedItem.dragging = false;
343+ }
341344 selectedItem = undefined;
345+ draggedIndex = -1;
342346 preDragging = false;
343347 postDragging = false;
344348 }
tests/qmltests/Launcher/tst_Launcher.qml+31
@@ -459,6 +459,37 @@ Item {
459459 waitUntilLauncherDisappears();
460460 }
461461
462+ function test_dragndrop_cancel() {
463+ dragLauncherIntoView();
464+ var draggedItem = findChild(launcher, "launcherDelegate4")
465+ var item0 = findChild(launcher, "launcherDelegate0")
466+ var fakeDragItem = findChild(launcher, "fakeDragItem")
467+
468+ // Doing longpress
469+ var currentMouseX = draggedItem.width / 2
470+ var currentMouseY = draggedItem.height / 2
471+ mousePress(draggedItem, currentMouseX, currentMouseY)
472+ // DraggedItem needs to hide and fakeDragItem become visible
473+ tryCompare(draggedItem, "itemOpacity", 0)
474+ tryCompare(fakeDragItem, "visible", true)
475+
476+ // Dragging
477+ currentMouseX -= units.gu(20)
478+ mouseMove(draggedItem, currentMouseX, currentMouseY)
479+
480+ // Make sure we're in the dragging state
481+ var dndArea = findChild(launcher, "dndArea");
482+ tryCompare(draggedItem, "dragging", true)
483+ tryCompare(dndArea, "draggedIndex", 4)
484+
485+ // Tap somewhere in the middle of the screen to close/hide the launcher
486+ tap(root)
487+
488+ // Make sure the dnd operation has been stopped
489+ tryCompare(draggedItem, "dragging", false)
490+ tryCompare(dndArea, "draggedIndex", -1)
491+ }
492+
462493 function test_quicklist_dismiss() {
463494 dragLauncherIntoView();
464495 var draggedItem = findChild(launcher, "launcherDelegate5")