fix: bind eraser to initial transform matrix

This fixes eraser being positioned against current object transform instead of initial object transform
fabricjs · Apr 7, 2021 · 302a23b · 302a23b
1 parent 48634eb
commit 302a23b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/brushes/eraser_brush.class.js b/src/brushes/eraser_brush.class.js
@@ -140,7 +140,8 @@
         inverted: true,
         dirty: true,
         eraser: mergedEraserPaths
-      })
+      });
+      obj.freezeClipPathTransformMatrix();
     },
 
     /**

diff --git a/src/shapes/object.class.js b/src/shapes/object.class.js
@@ -1172,6 +1172,14 @@
       return !!this.shadow && (this.shadow.offsetX !== 0 || this.shadow.offsetY !== 0);
     },
 
+    /**
+     * Used by @class fabric.EraserBrush to position the erased paths
+     */
+    _clipPathTransformMatrix: null,
+    freezeClipPathTransformMatrix: function () {
+      this._clipPathTransformMatrix = this.calcTransformMatrix();
+    },
+
     /**
      * Execute the drawing operation for an object clipPath
      * @param {CanvasRenderingContext2D} ctx Context to render on
@@ -1189,7 +1197,7 @@
       }
       //ctx.scale(1 / 2, 1 / 2);
       if (path.absolutePositioned) {
-        var m = fabric.util.invertTransform(this.calcTransformMatrix());
+        var m = fabric.util.invertTransform(this._clipPathTransformMatrix || this.calcTransformMatrix());
         ctx.transform(m[0], m[1], m[2], m[3], m[4], m[5]);
       }
       path.transform(ctx);