Add deg and fr as exceptions for 0 value unit omission (#2578)

* Add deg and fr as exceptions for 0 value unit omission * Make if statement more concise * Add unit tests for deg and fr
stylus · Sep 1, 2021 · 7334567 · 7334567
1 parent 57480a4
commit 7334567
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/lib/visitor/compiler.js b/lib/visitor/compiler.js
@@ -413,8 +413,8 @@ Compiler.prototype.visitUnit = function(unit){
 
   // Compress
   if (this.compress) {
-    // Always return '0' unless the unit is a percentage or time
-    if ('%' != type && 's' != type && 'ms' != type && 0 == n) return '0';
+    // Always return '0' unless the unit is a percentage, time, degree or fraction
+    if (!(['%', 's', 'ms', 'deg', 'fr'].includes(type)) && 0 == n) return '0';
     // Omit leading '0' on floats
     if (float && n < 1 && n > -1) {
       return n.toString().replace('0.', '.') + type;

diff --git a/test/cases/compress.units.css b/test/cases/compress.units.css
diff --git a/test/cases/compress.units.styl b/test/cases/compress.units.styl
@@ -19,3 +19,13 @@ body
 body
   foo 0s
   foo 0ms
+
+body
+  foo 0deg
+  foo 15deg
+  foo -15deg
+
+body
+  foo 0fr
+  foo 15fr
+  foo -15fr