Merge pull request #85 from yappbox/bug/hammer-cleanup

jasonbekolay · web-flow · commit 326366676eda · 2023-11-29T23:41:57.000-06:00
Set hammer instance to null after destroying so consumers know not to call it
diff --git a/addon/components/scroll-view.js b/addon/components/scroll-view.js
@@ -291,6 +291,7 @@ class ScrollView extends Component {
   unbindScrollerEvents(element) {
     if (this.hammer) {
       this.hammer.destroy();
+      this.hammer = null;
     }
     element.removeEventListener(
       normalizeWheel.getEventType(),

Original file line number	Diff line number	Diff line change
`@@ -291,6 +291,7 @@ class ScrollView extends Component {`
`291`	`291`	`unbindScrollerEvents(element) {`
`292`	`292`	`if (this.hammer) {`
`293`	`293`	`this.hammer.destroy();`
	`294`	`+ this.hammer = null;`
`294`	`295`	`}`
`295`	`296`	`element.removeEventListener(`
`296`	`297`	`normalizeWheel.getEventType(),`