Merge pull request #53 from postcss/sass-interpolation

Stop interpreting Sass interpolation as id; closes #52

Author: ben-eb

CHANGELOG.md

@@ -1,3 +1,7 @@
+# Head
+
+* Fixes misinterpretation of Sass interpolation (e.g. `#{foo}`) as an id selector.
+
 # 2.0.0
 
 This release contains the following breaking changes:

src/__tests__/id.js

@@ -39,3 +39,15 @@ test('extraneous non-combinating whitespace', '  #h1   ,  #h2   ', (t, tree) =>
     t.same(tree.nodes[1].nodes[0].spaces.before, '  ');
     t.same(tree.nodes[1].nodes[0].spaces.after, '   ');
 });
+
+test('Sass interpolation within a class', '.#{foo}', (t, tree) => {
+    t.same(tree.nodes[0].nodes.length, 1);
+    t.same(tree.nodes[0].nodes[0].type, 'class');
+    t.same(tree.nodes[0].nodes[0].value, '#{foo}');
+});
+
+test('Sass interpolation within an id', '#foo#{bar}', (t, tree) => {
+    t.same(tree.nodes[0].nodes.length, 1);
+    t.same(tree.nodes[0].nodes[0].type, 'id');
+    t.same(tree.nodes[0].nodes[0].value, 'foo#{bar}');
+});

src/parser.js

@@ -157,11 +157,11 @@ export default class Parser {
     error (message) {
         throw new this.input.error(message); // eslint-disable-line new-cap
     }
-    
+
     missingParenthesis () {
         return this.error('Expected opening parenthesis.');
     }
-    
+
     missingSquareBracket () {
         return this.error('Expected opening square bracket.');
     }
@@ -176,7 +176,7 @@ export default class Parser {
             return this.universal(before);
         }
     }
-    
+
     nesting () {
         this.newNode(new Nesting({
             value: this.currToken[1],
@@ -355,6 +355,11 @@ export default class Parser {
         }
         let hasClass = indexesOf(word, '.');
         let hasId = indexesOf(word, '#');
+        // Eliminate Sass interpolations from the list of id indexes
+        let interpolations = indexesOf(word, '#{');
+        if (interpolations.length) {
+            hasId = hasId.filter(hashIndex => !~interpolations.indexOf(hashIndex));
+        }
         let indices = sortAsc(uniq(flatten([[0], hasClass, hasId])));
         indices.forEach((ind, i) => {
             let index = indices[i + 1] || word.length;