Fixes #917 - prevents grid area unquoting.

Why:

* Specification requires such values to be quoted so should the values
  be preserved.

diff --git a/History.md b/History.md
index 2af83c3..1bcc55e 100644 (file)
--- a/History.md
+++ b/History.md
@@ -7,6 +7,11 @@
 * Fixed issue [#890](https://github.com/jakubpawlowicz/clean-css/issues/890) - adds toggle to disable empty tokens removal.
 * Fixed issue [#886](https://github.com/jakubpawlowicz/clean-css/issues/886) - better multi pseudo class / element merging.
 
+[4.0.10 / 2017-xx-xx](https://github.com/jakubpawlowicz/clean-css/compare/v4.0.9...4.0)
+==================
+
+* Fixed issue [#917](https://github.com/jakubpawlowicz/clean-css/issues/917) - prevents grid area unquoting.
+
 [4.0.9 / 2017-03-15](https://github.com/jakubpawlowicz/clean-css/compare/v4.0.8...v4.0.9)
 ==================
 

diff --git a/lib/optimizer/level-1/optimize.js b/lib/optimizer/level-1/optimize.js
index 9c0b65b..76d4f06 100644 (file)
--- a/lib/optimizer/level-1/optimize.js
+++ b/lib/optimizer/level-1/optimize.js
@@ -390,7 +390,7 @@ function optimizeZeroUnits(name, value) {
 }
 
 function removeQuotes(name, value) {
-  if (name == 'content' || name.indexOf('font-feature-settings') > -1) {
+  if (name == 'content' || name.indexOf('font-feature-settings') > -1 || name.indexOf('grid-') > -1) {
     return value;
   }
 

diff --git a/test/optimizer/level-1/optimize-test.js b/test/optimizer/level-1/optimize-test.js
index 6f66d93..5a5dd66 100644 (file)
--- a/test/optimizer/level-1/optimize-test.js
+++ b/test/optimizer/level-1/optimize-test.js
@@ -1136,6 +1136,22 @@ vows.describe('level 1 optimizations')
       '-webkit-font-feature-settings': [
         '.block{-webkit-font-feature-settings:"scmp","swsh" 2}',
         '.block{-webkit-font-feature-settings:"scmp","swsh" 2}'
+      ],
+      'grid-template': [
+        '.block{grid-template:"header" 20% "nav" auto}',
+        '.block{grid-template:"header" 20% "nav" auto}'
+      ],
+      'grid-template-areas - single word': [
+        '.block{grid-template-areas:"header" "nav" "body"}',
+        '.block{grid-template-areas:"header" "nav" "body"}'
+      ],
+      'grid-template-areas - multiple words': [
+        '.block{grid-template-areas:"header header" "nav" "body"}',
+        '.block{grid-template-areas:"header header" "nav" "body"}'
+      ],
+      'vendor prefixed grid-template': [
+        '.block{-ms-grid-template:"header" 20% "nav" auto}',
+        '.block{-ms-grid-template:"header" 20% "nav" auto}'
       ]
     }, { level: 1 })
   )