remove more events

fabricjs · Jun 3, 2018 · 061d047 · 061d047
1 parent 4af1c0d
commit 061d047
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js
@@ -122,6 +122,11 @@
       removeListener(this.upperCanvasEl, 'dragenter', this._onDragEnter);
       removeListener(this.upperCanvasEl, 'dragleave', this._onDragLeave);
       removeListener(this.upperCanvasEl, 'drop', this._onDrop);
+      // if you dispose on a mouseDown, before mouse up, you need to clean document to...
+      removeListener(fabric.document, 'mouseup', this._onMouseUp);
+      removeListener(fabric.document, 'touchend', this._onMouseUp, addEventOptions);
+      removeListener(fabric.document, 'mousemove', this._onMouseMove, addEventOptions);
+      removeListener(fabric.document, 'touchmove', this._onMouseMove, addEventOptions);
 
       if (typeof eventjs !== 'undefined' && 'remove' in eventjs) {
         eventjs.remove(this.upperCanvasEl, 'gesture', this._onGesture);