Fix an issue with space compression in calc values.

pieter-lazzaro · Sep 17, 2015 · 3c3bbb8 · 3c3bbb8
1 parent 619bdd8
commit 3c3bbb8
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,8 @@
+# 3.0.2
+
+* Fixes an issue where cssnano was removing spaces around forward slashes in
+  calc functions.
+
 # 3.0.1
 
 * Replaced css-list & balanced-match with postcss-value-parser, reducing the

diff --git a/lib/core.js b/lib/core.js
@@ -15,7 +15,10 @@ module.exports = postcss.plugin('cssnano-core', function () {
                 declaration.raws.important = '!important';
             }
             // Trim unnecessary space around e.g. 12px / 18px
-            declaration.value = declaration.value.replace(/\s*(\/)\s*/, '$1');
+            // TODO: This needs proper parsing.
+            if (!~declaration.value.indexOf('calc')) {
+                declaration.value = declaration.value.replace(/\s*(\/)\s*/, '$1');
+            }
             if (~[
                     'outline',
                     'outline-left',

diff --git a/tests/fixtures/preserveCalcSpaces.expected.css b/tests/fixtures/preserveCalcSpaces.expected.css
@@ -0,0 +1 @@
+div{width:calc(100vw / 2 - 6px + 0)}
diff --git a/tests/fixtures/preserveCalcSpaces.fixture.css b/tests/fixtures/preserveCalcSpaces.fixture.css
@@ -0,0 +1,3 @@
+div {
+    width: calc(100vw / 2 - 6px + 0);
+}