sass 3.2.0.alpha.258 → 3.2.0.alpha.259

data/REVISION

@@ -1 +1 @@
-77bf74d8fe5dc1ee1301f5e06676ae8581ddfd73
+389e108b5727c26e31d93a25ea7f0115fe270954

data/VERSION

@@ -1 +1 @@
-3.2.0.alpha.258
+3.2.0.alpha.259

data/lib/sass/script/functions.rb

@@ -144,9 +144,6 @@ module Sass::Script
   # \{#join join($list1, $list2, \[$separator\])}
   # : Joins together two lists into one.
   #
-  # \{#append append($list1, $val, \[$separator\])}
-  # : Appends a single value onto the end of a list.
-  #
   # ## Introspection Functions
   #
   # \{#type_of type-of($value)}
@@ -585,10 +582,7 @@ module Sass::Script
         return Sass::Script::String.new("alpha(#{args.map {|a| a.to_s}.join(", ")})")
       end
 
-      raise ArgumentError.new("wrong number of arguments (#{args.size} for 1)") if args.size != 1
-
-      assert_type args.first, :Color
-      Sass::Script::Number.new(args.first.alpha)
+      opacity(*args)
     end
     declare :alpha, [:color]
 
@@ -601,7 +595,6 @@ module Sass::Script
     # @see #transparentize
     # @raise [ArgumentError] If `color` isn't a color
     def opacity(color)
-      return Sass::Script::String.new("opacity(#{color})") if color.is_a?(Sass::Script::Number)
       assert_type color, :Color
       Sass::Script::Number.new(color.alpha)
     end
@@ -692,21 +685,16 @@ module Sass::Script
     # @example
     #   saturate(hsl(120, 30%, 90%), 20%) => hsl(120, 50%, 90%)
     #   saturate(#855, 20%) => #9e3f3f
-    # @overload saturate(color, amount)
-    #   @param color [Color]
-    #   @param amount [Number]
-    #   @return [Color]
-    #   @see #desaturate
-    #   @raise [ArgumentError] If `color` isn't a color,
-    #     or `number` isn't a number between 0% and 100%
-    def saturate(color, amount = nil)
-      # Support the filter effects definition of saturate.
-      # https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html
-      return Sass::Script::String.new("saturate(#{color})") if amount.nil?
+    # @param color [Color]
+    # @param amount [Number]
+    # @return [Color]
+    # @see #desaturate
+    # @raise [ArgumentError] If `color` isn't a color,
+    #   or `number` isn't a number between 0% and 100%
+    def saturate(color, amount)
       _adjust(color, amount, :saturation, 0..100, :+, "%")
     end
     declare :saturate, [:color, :amount]
-    declare :saturate, [:amount]
 
     # Makes a color less saturated.
     # Takes a color and an amount between 0% and 100%,
@@ -1007,13 +995,21 @@ module Sass::Script
     declare :mix, [:color_1, :color_2]
     declare :mix, [:color_1, :color_2, :weight]
 
-    # Converts a color to grayscale.
-    # This is identical to `desaturate(color, 100%)`.
+    # @overload grayscale(color)
+    #   Converts a color to grayscale.
+    #   This is identical to `desaturate(color, 100%)`.
     #
-    # @param color [Color]
-    # @return [Color]
-    # @raise [ArgumentError] if `color` isn't a color
-    # @see #desaturate
+    #   @param color [Color]
+    #   @return [Color]
+    #   @raise [ArgumentError] if `color` isn't a color
+    #   @see #desaturate
+    # @overload grayscale(number)
+    #   Returns an unquoted string `grayscale(number)`, as though the function
+    #   were not defined. This is for the `grayscale` function used in
+    #   `-webkit-filter`.
+    #
+    #   @param number [Number]
+    #   @return [Sass::Script::String]
     def grayscale(color)
       return Sass::Script::String.new("grayscale(#{color})") if color.is_a?(Sass::Script::Number)
       desaturate color, Number.new(100)
@@ -1039,8 +1035,6 @@ module Sass::Script
     # @return [Color]
     # @raise [ArgumentError] if `color` isn't a color
     def invert(color)
-      return Sass::Script::String.new("invert(#{color})") if color.is_a?(Sass::Script::Number)
-
       assert_type color, :Color
       color.with(
         :red => (255 - color.red),
@@ -1334,14 +1328,14 @@ module Sass::Script
     #   append(10px 20px, 30px) => 10px 20px 30px
     #   append((blue, red), green) => blue, red, green
     #   append(10px 20px, 30px 40px) => 10px 20px (30px 40px)
-    #   append(10px, 20px, comma) => 10px, 20px
-    #   append((blue, red), green, space) => blue red green
-    # @overload append(list, val, separator: auto)
-    #   @param list [Literal] The list to add the value to
-    #   @param val [Literal] The value to add to the end of the list
+    #   join(10px, 20px, comma) => 10px, 20px
+    #   join((blue, red), green, space) => blue red green
+    # @overload join(list, val, separator: auto)
+    #   @param list1 [Literal] The first list to join
+    #   @param list2 [Literal] The second list to join
     #   @param separator [String] How the list separator (comma or space) should be determined.
     #     If this is `comma` or `space`, that is always the separator;
-    #     if this is `auto` (the default), the separator is the same as that used by the list.
+    #     if this is `auto` (the default), the separator is determined as explained above.
     def append(list, val, separator = Sass::Script::String.new("auto"))
       assert_type separator, :String
       unless %w[auto space comma].include?(separator.value)

data/test/sass/functions_test.rb

@@ -336,18 +336,6 @@ class SassFunctionTest < Test::Unit::TestCase
     assert_error_message("12 is not a color for `alpha'", "alpha(12)")
   end
 
-  def test_opacity
-    assert_equal("1", evaluate("opacity(#123456)"))
-    assert_equal("0.34", evaluate("opacity(rgba(0, 1, 2, 0.34))"))
-    assert_equal("0", evaluate("opacity(hsla(0, 1, 2, 0))"))
-    assert_equal("0", evaluate("opacity($color: hsla(0, 1, 2, 0))"))
-    assert_equal("opacity(20%)", evaluate("opacity(20%)"))
-  end
-
-  def test_opacity_exception
-    assert_error_message("\"foo\" is not a color for `opacity'", "opacity(foo)")
-  end
-
   def test_opacify
     assert_equal("rgba(0, 0, 0, 0.75)", evaluate("opacify(rgba(0, 0, 0, 0.5), 0.25)"))
     assert_equal("rgba(0, 0, 0, 0.3)", evaluate("opacify(rgba(0, 0, 0, 0.2), 0.1)"))
@@ -447,7 +435,6 @@ class SassFunctionTest < Test::Unit::TestCase
     assert_equal("#88aa88", evaluate("saturate(#8a8, 0%)"))
     assert_equal("rgba(158, 63, 63, 0.5)", evaluate("saturate(rgba(136, 85, 85, 0.5), 20%)"))
     assert_equal("rgba(158, 63, 63, 0.5)", evaluate("saturate($color: rgba(136, 85, 85, 0.5), $amount: 20%)"))
-    assert_equal("saturate(50%)", evaluate("saturate(50%)"))
   end
 
   def test_saturate_tests_bounds
@@ -841,7 +828,6 @@ class SassFunctionTest < Test::Unit::TestCase
   def test_invert
     assert_equal("#112233", evaluate("invert(#edc)"))
     assert_equal("rgba(245, 235, 225, 0.5)", evaluate("invert(rgba(10, 20, 30, 0.5))"))
-    assert_equal("invert(20%)", evaluate("invert(20%)"))
   end
 
   def test_invert_tests_types

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification 
 name: sass
 version: !ruby/object:Gem::Version 
-  hash: 592302361
+  hash: 592302363
   prerelease: 6
   segments: 
   - 3
   - 2
   - 0
   - alpha
-  - 258
-  version: 3.2.0.alpha.258
+  - 259
+  version: 3.2.0.alpha.259
 platform: ruby
 authors: 
 - Nathan Weizenbaum