fix(convertPathData): fix q control point behavior when item is remov…

…ed (#1927)
svg · Jan 6, 2024 · 6afc7bd · 6afc7bd
1 parent 9e01769
commit 6afc7bd
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/plugins/convertPathData.js b/plugins/convertPathData.js
@@ -394,7 +394,6 @@ function filters(
 
   path = path.filter(function (item, index, path) {
     const qControlPoint = prevQControlPoint;
-    prevQControlPoint = undefined;
 
     let command = item.command;
     let data = item.args;
@@ -895,6 +894,8 @@ function filters(
         // @ts-ignore
         prevQControlPoint = item.coords;
       }
+    } else {
+      prevQControlPoint = undefined;
     }
     prev = item;
     return true;

diff --git a/test/plugins/convertPathData.36.svg.txt b/test/plugins/convertPathData.36.svg.txt
@@ -0,0 +1,15 @@
+Should process/optimize q correctly even if useless commands in between.
+
+See: https://github.com/svg/svgo/issues/1926
+
+===
+
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 1 20">
+  <path d="M-6.3 9.9q.7-4.5.2-5-.5-.5-1.5-.5l0 0q-.4 0-2 .3"/>
+</svg>
+
+@@@
+
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 1 20">
+    <path d="M-6.3 9.9q.7-4.5.2-5t-1.5-.5q-.4 0-2 .3"/>
+</svg>