fix(postcss-convert-values): preserve percent sign in length-percentage syntax (#1702)

Multiply · web-flow · commit 27710c5cc54a · 2025-05-12T16:11:24.000+02:00
diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md
@@ -73,3 +73,4 @@ Thanks goes to these wonderful people:
 - Seth Falco
 - Seiya
 - Aleksandr Kondrashov
+- Jens Fosgerau
diff --git a/packages/postcss-convert-values/src/index.js b/packages/postcss-convert-values/src/index.js
@@ -157,7 +157,10 @@ function shouldKeepZeroUnit(decl, browsers) {
         (node) =>
           node.type === 'decl' &&
           node.prop.toLowerCase() === 'syntax' &&
-          (node.value === "'<percentage>'" || node.value === '"<percentage>"')
+          (node.value === "'<percentage>'" ||
+            node.value === '"<percentage>"' ||
+            node.value === "'<length-percentage>'" ||
+            node.value === '"<length-percentage>"')
       )) ||
     keepWhenZero.has(lowerCasedProp)
   );
diff --git a/packages/postcss-convert-values/test/index.js b/packages/postcss-convert-values/test/index.js
@@ -512,6 +512,22 @@ test(
   )
 );
 
+test(
+  `should not strip the percentage from 0 in @property, for initial-value (length-percentage)`,
+  processCSS(
+    `@property --percent{syntax:'<length-percentage>';inherits:false;initial-value:0%;}`,
+    `@property --percent{syntax:'<length-percentage>';inherits:false;initial-value:0%;}`
+  )
+);
+
+test(
+  `should not strip the percentage from 0 in @property, for initial-value (length-percentage, syntax string in double quotes)`,
+  processCSS(
+    `@property --percent{syntax:"<length-percentage>";inherits:false;initial-value:0%;}`,
+    `@property --percent{syntax:"<length-percentage>";inherits:false;initial-value:0%;}`
+  )
+);
+
 test(
   'should not strip the percentage from background-color',
   passthroughCSS('background-color:color-mix(#000, #FFF 0%);')
diff --git a/packages/postcss-convert-values/types/index.d.ts.map b/packages/postcss-convert-values/types/index.d.ts.map
