Fix for multiple consequent spaces present in CSS selectors, fixes #2670

Polymer · Nov 3, 2015 · ecddb56 · ecddb56
1 parent 0dc69df
commit ecddb56
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/src/lib/css-parse.html b/src/lib/css-parse.html
@@ -60,6 +60,7 @@
       if (node.parent) {
         var ss = node.previous ? node.previous.end : node.parent.start;
         t = text.substring(ss, node.start-1);
+        t = t.replace(this._rx.multipleSpaces, ' ');
         // TODO(sorvell): ad hoc; make selector include only after last ;
         // helps with mixin syntax
         t = t.substring(t.lastIndexOf(';')+1);
@@ -162,6 +163,7 @@
       mixinApply: /@apply[\s]*\([^)]*?\)[\s]*(?:[;\n]|$)?/gim,
       varApply: /[^;:]*?:[^;]*var[^;]*(?:[;\n]|$)?/gim,
       keyframesRule: /^@[^\s]*keyframes/,
+      multipleSpaces: /\s+/g
     },
 
     VAR_START: '--',

diff --git a/test/unit/css-parse.html b/test/unit/css-parse.html
@@ -60,6 +60,15 @@
     }
     /* comment */
   </style>
+
+  <style id="multiple-spaces">
+    .foo  .bar {}
+    .foo    .bar {}
+    .foo
+
+
+    .bar {}
+  </style>
 <script>
 
   function sanitizeCss(text) {
@@ -119,6 +128,14 @@
       assert.equal(result, '.stuff { background: red; }', 'unexpected stringified output');
     });
 
+    test('multiple consequent spaces in CSS selector', function() {
+      var s4 = document.querySelector('#multiple-spaces');
+      var t = css.parse(s4.textContent);
+      assert.equal(t.rules[0].selector, '.foo .bar');
+      assert.equal(t.rules[1].selector, '.foo .bar');
+      assert.equal(t.rules[2].selector, '.foo .bar');
+    });
+
   });
 </script>