Skip to content

Commit 3b50536

Browse files
committed
fix: add overContainer property to dragOutEvent
1 parent 81c697c commit 3b50536

File tree

2 files changed

+26
-0
lines changed

2 files changed

+26
-0
lines changed

src/Draggable/Draggable.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -477,6 +477,7 @@ export default class Draggable {
477477
sourceContainer: container,
478478
sensorEvent,
479479
over: this.currentOver,
480+
overContainer: this.currentOverContainer,
480481
});
481482

482483
this.currentOver.classList.remove(this.getClassNameFor('draggable:over'));

src/Draggable/tests/Draggable.test.js

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -927,4 +927,29 @@ describe('Draggable', () => {
927927
releaseMouse(newInstance.source);
928928
});
929929
});
930+
931+
describe('when `drag:out`', () => {
932+
it('should trigger dragOutEvent', () => {
933+
const newInstance = new Draggable(containers, {
934+
draggable: 'li',
935+
});
936+
const draggableElement = sandbox.querySelector('li');
937+
document.elementFromPoint = () => draggableElement;
938+
939+
newInstance.on('drag:out', (event) => {
940+
expect(event.overContainer).toBe(containers[0]);
941+
});
942+
943+
clickMouse(draggableElement);
944+
945+
waitForDragDelay();
946+
947+
document.elementFromPoint = () => draggableElement.nextElementSibling;
948+
moveMouse(draggableElement.nextElementSibling);
949+
950+
waitForDragDelay();
951+
952+
releaseMouse(newInstance.source);
953+
});
954+
});
930955
});

0 commit comments

Comments
 (0)