Undesired spaces with partial reference selector using ranges. Fixes s…

…tylus#2133
Zearin · Mar 5, 2016 · 2bd852e · 2bd852e
1 parent e4ca920
commit 2bd852e
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 3 deletions.
diff --git a/lib/selector-parser.js b/lib/selector-parser.js
@@ -190,7 +190,9 @@ SelectorParser.prototype.number = function() {
 
 SelectorParser.prototype.range = function() {
   var start = this.number()
+    , parsed
     , ret;
+
   if ('..' == this.str.slice(0, 2)) {
     this.skip(2);
     var end = this.number()
@@ -206,11 +208,17 @@ SelectorParser.prototype.range = function() {
     }
 
     if (end < len - 1) {
-      ret = this.parts.slice(start, end + 1).map(function(part) {
+      parsed = this.parts.slice(start, end + 1).map(function(part) {
         var selector = new SelectorParser(part, this.stack, this.parts);
         selector.raw = true;
-        return selector.parse().val.trim();
-      }, this).join(' ');
+        return selector.parse();
+      }, this);
+
+      ret = parsed.map(function(selector) {
+        return selector.val.trim();
+      }).join(parsed.every(function(selector) {
+        return selector.nested;
+      }) ? ' ' : '');
     }
   } else {
     ret = this.stack[

diff --git a/test/cases/selector.reference.css b/test/cases/selector.reference.css
@@ -105,3 +105,9 @@ foo /deep/ /deep/ {
   test: '.foo';
   test: '.baz ~/';
 }
+.s1-s2-s3 {
+  syntax: "^[2]";
+}
+.s1-s2-s3 {
+  syntax: "^[0..2]";
+}
diff --git a/test/cases/selector.reference.styl b/test/cases/selector.reference.styl
@@ -127,3 +127,14 @@ foo /deep/
     ~/
       test: selector('~/')
       test: selector('.baz ~/')
+
+.s1
+  &-s2
+    &-s3
+      &-s4
+        ^[2]
+          // first three levels of selectors
+          syntax: "^[2]"
+        ^[0..2]
+          // I expect this to be identical to the above, but it's not
+          syntax: "^[0..2]"