Merge pull request #586 from clauderic/fix-mobile-listener-node-issue

clauderic · web-flow · commit 0049d0d89f30 · 2021-03-18T22:23:26.000-04:00
Fix touch events being lost by listening to event.target on mobile
diff --git a/src/SortableContainer/index.js b/src/SortableContainer/index.js
@@ -392,7 +392,7 @@ export default function sortableContainer(
             .forEach((className) => this.helper.classList.add(className));
         }
 
-        this.listenerNode = event.touches ? node : this.contentWindow;
+        this.listenerNode = event.touches ? event.target : this.contentWindow;
 
         if (isKeySorting) {
           this.listenerNode.addEventListener('wheel', this.handleKeyEnd, true);

Original file line number	Diff line number	Diff line change
`@@ -392,7 +392,7 @@ export default function sortableContainer(`
`392`	`392`	`.forEach((className) => this.helper.classList.add(className));`
`393`	`393`	`}`
`394`	`394`
`395`		`- this.listenerNode = event.touches ? node : this.contentWindow;`
	`395`	`+ this.listenerNode = event.touches ? event.target : this.contentWindow;`
`396`	`396`
`397`	`397`	`if (isKeySorting) {`
`398`	`398`	`this.listenerNode.addEventListener('wheel', this.handleKeyEnd, true);`