haml-edge 2.3.218 → 2.3.219

data/EDGE_GEM_VERSION

@@ -1 +1 @@
-2.3.218
+2.3.219

data/VERSION

@@ -1 +1 @@
-2.3.218
+2.3.219

data/lib/sass/selector/sequence.rb

@@ -79,11 +79,19 @@ module Sass
       def do_extend(extends, supers = [])
         paths = Haml::Util.paths(members.map do |sseq_or_op|
             next [[sseq_or_op]] unless sseq_or_op.is_a?(SimpleSequence)
-            [[sseq_or_op], *sseq_or_op.do_extend(extends, supers).map {|seq| seq.members}]
+            extended = sseq_or_op.do_extend(extends, supers)
+            choices = extended.map {|seq| seq.members}
+            choices.unshift([sseq_or_op]) unless extended.any? {|seq| seq.superselector?(sseq_or_op)}
+            choices
           end)
         Haml::Util.flatten(paths.map {|path| weave(path)}, 1).map {|p| Sequence.new(p)}
       end
 
+      def superselector?(sseq)
+        return false unless members.size == 1
+        members.last.superselector?(sseq)
+      end
+
       # @see Simple#to_a
       def to_a
         ary = @members.map {|seq_or_op| seq_or_op.is_a?(SimpleSequence) ? seq_or_op.to_a : seq_or_op}

data/lib/sass/selector/simple_sequence.rb

@@ -62,21 +62,18 @@ module Sass
       #   by extending this selector with `extends`.
       # @see CommaSequence#do_extend
       def do_extend(extends, supers = [])
-        seqs = extends.get(members.to_set).map do |seq, sels|
+        extends.get(members.to_set).map do |seq, sels|
           # If A {@extend B} and C {...},
           # seq is A, sels is B, and self is C
 
           self_without_sel = self.members - sels
           next unless unified = seq.members.last.unify(self_without_sel)
           [sels, seq.members[0...-1] + [unified]]
-        end.compact.map {|sels, seq| [sels, Sequence.new(seq)]}
-
-        seqs.map {|_, seq| seq}.concat(
-          seqs.map do |sels, seq|
-            new_seqs = seq.do_extend(extends, supers.unshift(sels))[1..-1]
-            supers.shift
-            new_seqs
-          end.flatten.uniq)
+        end.compact.map {|sels, seq| [sels, Sequence.new(seq)]}.map do |sels, seq|
+          seqs = seq.do_extend(extends, supers.unshift(sels))
+          supers.shift
+          seqs
+        end.flatten.uniq
       rescue SystemStackError
         handle_extend_loop(supers)
       end
@@ -102,6 +99,10 @@ module Sass
         SimpleSequence.new(sseq)
       end
 
+      def superselector?(sseq)
+        (base.nil? || base.eql?(sseq.base)) && rest.subset?(sseq.rest)
+      end
+
       # @see Simple#to_a
       def to_a
         @members.map {|sel| sel.to_a}.flatten

data/test/sass/extend_test.rb

@@ -102,7 +102,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo.bar, .bar.baz, .foo.baz, .baz {
+.foo.bar, .baz {
   a: b; }
 CSS
 .foo.bar {a: b}
@@ -121,7 +121,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo.bar, .bar.baz, .foo.bang, .baz.bang {
+.foo.bar, .bar.baz, .baz.bang, .foo.bang {
   a: b; }
 CSS
 .foo.bar {a: b}
@@ -190,7 +190,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo.baz, .baz {
+.baz {
   a: b; }
 CSS
 .foo.baz {a: b}
@@ -208,7 +208,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo#baz, #baz {
+#baz {
   a: b; }
 CSS
 .foo#baz {a: b}
@@ -242,7 +242,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo.bar, .bar {
+.bar {
   a: b; }
 CSS
 .foo.bar {a: b}
@@ -250,7 +250,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-.foo.bar, .bar {
+.bar {
   a: b; }
 CSS
 .foo.bar {a: b}
@@ -268,7 +268,7 @@ SCSS
 
   def test_universal_unification_with_namespaceless_universal_target
     assert_equal <<CSS, render(<<SCSS)
-*.foo, * {
+* {
   a: b; }
 CSS
 *.foo {a: b}
@@ -276,7 +276,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-*.foo, * {
+* {
   a: b; }
 CSS
 *.foo {a: b}
@@ -292,7 +292,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-*|*.foo, *|* {
+*|* {
   a: b; }
 CSS
 *|*.foo {a: b}
@@ -318,7 +318,7 @@ SCSS
 
   def test_universal_unification_with_namespaced_universal_target
     assert_equal <<CSS, render(<<SCSS)
-ns|*.foo, ns|* {
+ns|* {
   a: b; }
 CSS
 ns|*.foo {a: b}
@@ -326,7 +326,7 @@ ns|*.foo {a: b}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|*.foo, ns|* {
+ns|* {
   a: b; }
 CSS
 ns|*.foo {a: b}
@@ -342,7 +342,7 @@ ns2|* {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|*.foo, ns|* {
+ns|* {
   a: b; }
 CSS
 ns|*.foo {a: b}
@@ -352,7 +352,7 @@ SCSS
 
   def test_universal_unification_with_namespaceless_element_target
     assert_equal <<CSS, render(<<SCSS)
-a.foo, a {
+a {
   a: b; }
 CSS
 a.foo {a: b}
@@ -360,7 +360,7 @@ a.foo {a: b}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-a.foo, a {
+a {
   a: b; }
 CSS
 a.foo {a: b}
@@ -376,7 +376,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-*|a.foo, *|a {
+*|a {
   a: b; }
 CSS
 *|a.foo {a: b}
@@ -402,7 +402,7 @@ SCSS
 
   def test_universal_unification_with_namespaced_element_target
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -410,7 +410,7 @@ ns|a.foo {a: b}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -426,7 +426,7 @@ ns2|* {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -554,7 +554,7 @@ SCSS
 
   def test_element_unification_with_namespaceless_element_target
     assert_equal <<CSS, render(<<SCSS)
-a.foo, a {
+a {
   a: b; }
 CSS
 a.foo {a: b}
@@ -562,7 +562,7 @@ a {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-a.foo, a {
+a {
   a: b; }
 CSS
 a.foo {a: b}
@@ -578,7 +578,7 @@ a {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-*|a.foo, *|a {
+*|a {
   a: b; }
 CSS
 *|a.foo {a: b}
@@ -612,7 +612,7 @@ SCSS
 
   def test_element_unification_with_namespaced_element_target
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -620,7 +620,7 @@ a {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -636,7 +636,7 @@ ns2|a {@extend .foo}
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-ns|a.foo, ns|a {
+ns|a {
   a: b; }
 CSS
 ns|a.foo {a: b}
@@ -678,7 +678,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-[foo=bar].baz, [foo=bar] {
+[foo=bar] {
   a: b; }
 CSS
 [foo=bar].baz {a: b}
@@ -720,7 +720,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-::foo.baz, ::foo {
+::foo {
   a: b; }
 CSS
 ::foo.baz {a: b}
@@ -728,7 +728,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-::foo(2n+1).baz, ::foo(2n+1) {
+::foo(2n+1) {
   a: b; }
 CSS
 ::foo(2n+1).baz {a: b}
@@ -744,7 +744,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-:foo.baz, :foo {
+:foo {
   a: b; }
 CSS
 :foo.baz {a: b}
@@ -844,7 +844,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-:not(.foo).baz, :not(.foo) {
+:not(.foo) {
   a: b; }
 CSS
 :not(.foo).baz {a: b}
@@ -852,7 +852,7 @@ CSS
 SCSS
 
     assert_equal <<CSS, render(<<SCSS)
-:not([a=b]).baz, :not([a=b]) {
+:not([a=b]) {
   a: b; }
 CSS
 :not([a=b]).baz {a: b}
@@ -1100,6 +1100,16 @@ CSS
 SCSS
   end
 
+  def test_multiple_extender_merges_with_superset_selector
+    assert_equal <<CSS, render(<<SCSS)
+a.bar.baz, a.foo {
+  a: b; }
+CSS
+.foo {@extend .bar; @extend .baz}
+a.bar.baz {a: b}
+SCSS
+  end
+
   private
 
   def render(sass, options = {})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: haml-edge
 version: !ruby/object:Gem::Version 
-  version: 2.3.218
+  version: 2.3.219
 platform: ruby
 authors: 
 - Nathan Weizenbaum