address review

svg · Nov 12, 2023 · c6c98cd · c6c98cd
1 parent 0234ccc
commit c6c98cd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/docs/03-plugins/convert-path-data.mdx b/docs/03-plugins/convert-path-data.mdx
@@ -19,7 +19,7 @@ svgo:
       description: If to convert regular lines to an explicit horizontal or vertical line where possible.
       default: true
     convertToZ:
-      description: If to convert lines that go to the start to a `z` command.
+      description: If to convert lines that go to the start to a <code>z</code> command.
       default: true
     curveSmoothShorthands:
       description: If to convert curves to smooth curves where possible.
@@ -58,7 +58,7 @@ This plugin uses multiple techniques to either reduce the number of instructions
 
 * Convert between relative or absolute coordinates, whichever is shortest.
 * Convert between commands. For example, a bézier curve that behaves like a straight line might as well use a line instruction.
-* Remove commands (when applicable). For example, a command that moves to the current position can be removed.
+* Remove redundant commands. For example, a command that moves to the current position can be removed.
 * Trim redundant delimiters and leading zeros.
 * Round numeric values using conventional rounding rules.
 

diff --git a/plugins/convertPathData.js b/plugins/convertPathData.js
@@ -170,8 +170,6 @@ exports.fn = (root, params) => {
             (computedStyle['stroke-linecap'].type === 'dynamic' ||
               computedStyle['stroke-linecap'].value !== 'butt');
           const maybeHasStrokeAndLinecap = maybeHasStroke && maybeHasLinecap;
-          // TODO: when stroke is used, prefer z more
-          // Z is the same as going home at 90 and 0 degrees, but not at other degrees
           const isSafeToUseZ = maybeHasStroke
             ? computedStyle['stroke-linecap']?.type === 'static' &&
               computedStyle['stroke-linecap'].value === 'round' &&