Merge pull request #693 from Cerdic/Issue/leafo/438

Issue/leafo/438

Author: robocoder

src/Compiler.php

@@ -1414,30 +1414,61 @@ protected function evalSelectorPart($part)
      * Collapse selectors
      *
      * @param array $selectors
+     * @param bool $selectorFormat
+     *   if false return a collapsed string
+     *   if true return an array description of a structured selector
      *
      * @return string
      */
-    protected function collapseSelectors($selectors)
+    protected function collapseSelectors($selectors, $selectorFormat = false)
     {
         $parts = [];
 
         foreach ($selectors as $selector) {
-            $output = '';
+            $output = [];
+            $glueNext = false;
             foreach ($selector as $node) {
-                $output .= ($output ? ' ' : '');
+                $compound = '';
 
                 array_walk_recursive(
                     $node,
-                    function ($value, $key) use (&$output) {
-                        $output .= $value;
+                    function ($value, $key) use (&$compound) {
+                        $compound .= $value;
                     }
                 );
+                if ($selectorFormat && $this->isImmediateRelationshipCombinator($compound)) {
+                    if (count($output)) {
+                        $output[count($output) - 1] .= ' ' . $compound;
+                    } else {
+                        $output[] = $compound;
+                    }
+                    $glueNext = true;
+                } elseif ($glueNext) {
+                    $output[count($output) - 1] .= ' ' . $compound;
+                    $glueNext = false;
+                } else {
+                    $output[] = $compound;
+                }
             }
 
+            if ($selectorFormat) {
+                foreach ($output as &$o) {
+                    $o = [Type::T_STRING, '', [$o]];
+                }
+                $output = [Type::T_LIST, ' ', $output];
+            } else {
+                $output = implode(' ', $output);
+            }
             $parts[] = $output;
         }
 
-        return implode(', ', $parts);
+        if ($selectorFormat) {
+            $parts = [Type::T_LIST, ',', $parts];
+        } else {
+            $parts = implode(', ', $parts);
+        }
+
+        return $parts;
     }
 
     /**
@@ -2638,9 +2669,8 @@ protected function reduce($value, $inExp = false)
 
             case Type::T_SELF:
                 $selfSelector = $this->multiplySelectors($this->env);
-                $selfSelector = $this->collapseSelectors($selfSelector);
-
-                return [Type::T_STRING, '', [$selfSelector]];
+                $selfSelector = $this->collapseSelectors($selfSelector, true);
+                return $selfSelector;
 
             default:
                 return $value;

tests/inputs/selector_functions.scss

@@ -0,0 +1,14 @@
+.foo, .bar, .foo2 .bar2, .foo3 > .bar3 {
+	@each $selector in & {
+		content: "#{$selector}";
+	}
+}
+
+
+.main aside:hover,
+.sidebar p {
+	@each $selector in & {
+		content: "#{$selector}";
+	}
+}
+

tests/outputs/selector_functions.css

@@ -0,0 +1,9 @@
+.foo, .bar, .foo2 .bar2, .foo3 > .bar3 {
+  content: ".foo";
+  content: ".bar";
+  content: ".foo2 .bar2";
+  content: ".foo3 > .bar3"; }
+
+.main aside:hover, .sidebar p {
+  content: ".main aside:hover";
+  content: ".sidebar p"; }

tests/outputs_numbered/selector_functions.css

@@ -0,0 +1,10 @@
+/* line 1, inputs/selector_functions.scss */
+.foo, .bar, .foo2 .bar2, .foo3 > .bar3 {
+  content: ".foo";
+  content: ".bar";
+  content: ".foo2 .bar2";
+  content: ".foo3 > .bar3"; }
+/* line 8, inputs/selector_functions.scss */
+.main aside:hover, .sidebar p {
+  content: ".main aside:hover";
+  content: ".sidebar p"; }