feat: accessor for mousewheel

should check status before update enable status, avoid bind event again and again re #16 #41
antvis · Dec 19, 2019 · 4537463 · 4537463
1 parent 42abf49
commit 4537463
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/packages/x6/src/graph/mousewheel-accessor.ts b/packages/x6/src/graph/mousewheel-accessor.ts
@@ -5,13 +5,26 @@ export class MouseWheelAccessor extends BaseGraph {
     return this.mouseWheelHandler.isEnabled()
   }
 
+  setMouseWheelEnabled(enabled: boolean) {
+    if (enabled) {
+      this.enableMouseWheel()
+    } else {
+      this.disableMouseWheel()
+    }
+    return this
+  }
+
   enableMouseWheel() {
-    this.mouseWheelHandler.enable()
+    if (!this.isMouseWheelEnabled()) {
+      this.mouseWheelHandler.enable()
+    }
     return this
   }
 
   disableMouseWheel() {
-    this.mouseWheelHandler.disable()
+    if (this.isMouseWheelEnabled()) {
+      this.mouseWheelHandler.disable()
+    }
     return this
   }