Fix decimante points breaking point drawing in freedrawing (#5771)

fabricjs · Jun 22, 2019 · e826898 · e826898
1 parent 8a6e683
commit e826898
Show file tree

Hide file tree

Showing 4 changed files with 57 additions and 2 deletions.
diff --git a/src/brushes/pencil_brush.class.js b/src/brushes/pencil_brush.class.js
@@ -110,7 +110,7 @@
      * @private
      */
     _reset: function() {
-      this._points.length = 0;
+      this._points = [];
       this._setBrushStyles();
       this._setShadow();
     },
@@ -240,6 +240,9 @@
           newPoints.push(lastPoint);
         }
       }
+      if (newPoints.length === 1) {
+        newPoints.push(new fabric.Point(newPoints[0].x, newPoints[0].y));
+      }
       return newPoints;
     },
 

diff --git a/test/visual/freedraw.js b/test/visual/freedraw.js
diff --git a/test/visual/golden/freedrawing5.png b/test/visual/golden/freedrawing5.png
diff --git a/test/visual/golden/freedrawing6.png b/test/visual/golden/freedrawing6.png