Fix empty path segments removing with stroke-linecap

Fix svg#1017
mbrukman · Sep 16, 2018 · 831f95f · 831f95f
1 parent a165206
commit 831f95f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/plugins/convertPathData.js b/plugins/convertPathData.js
@@ -36,7 +36,8 @@ var pathElems = require('./_collections.js').pathElems,
     error,
     arcThreshold,
     arcTolerance,
-    hasMarkerMid;
+    hasMarkerMid,
+    hasStrokeLinecap;
 
 /**
  * Convert absolute Path to relative,
@@ -67,6 +68,10 @@ exports.fn = function(item, params) {
         }
         hasMarkerMid = item.hasAttr('marker-mid');
 
+        var stroke = item.computedAttr('stroke'),
+            strokeLinecap = item.computedAttr('stroke');
+        hasStrokeLinecap = stroke && stroke != 'none' && strokeLinecap && strokeLinecap != 'butt';
+
         var data = path2js(item);
 
         // TODO: get rid of functions returns
@@ -584,7 +589,7 @@ function filters(path, params) {
             }
 
             // remove useless non-first path segments
-            if (params.removeUseless) {
+            if (params.removeUseless && !hasStrokeLinecap) {
 
                 // l 0,0 / h 0 / v 0 / q 0,0 0,0 / t 0,0 / c 0,0 0,0 0,0 / s 0,0 0,0
                 if (

diff --git a/test/plugins/convertPathData.03.svg b/test/plugins/convertPathData.03.svg