compass 0.11.beta.5 → 0.11.beta.6

data/VERSION.yml

@@ -2,4 +2,4 @@
 :major: 0
 :minor: 11
 :state: beta
-:build: 5
+:build: 6

data/frameworks/compass/stylesheets/compass/css3/_images.scss

@@ -16,15 +16,36 @@
   $backgrounds: compact($background-1, $background-2, $background-3, $background-4, $background-5,
                         $background-6, $background-7, $background-8, $background-9, $background-10);
   $mult-bgs: -compass-list-size($backgrounds) > 1;
-  $simple-background: if($mult-bgs or prefixed(-css2, $backgrounds), -css2(-compass-nth($backgrounds, last)), false);
-  @if not blank($simple-background)                                                       {       background:    $simple-background; }
   @if $experimental-support-for-svg     and prefixed(-svg,    $backgrounds)               {       background:    -svg($backgrounds); }
   @if $experimental-support-for-webkit  and prefixed(-webkit, $backgrounds)               {       background: -webkit($backgrounds); }
   @if $experimental-support-for-mozilla and prefixed(-moz,    $backgrounds)               {       background:    -moz($backgrounds); }
+  @if $experimental-support-for-opera   and prefixed(-o,      $backgrounds)               {       background:      -o($backgrounds); }
   @if $experimental-support-for-pie     and (prefixed(-pie,   $backgrounds) or $mult-bgs) {  -pie-background:    -pie($backgrounds); }
                                                                                                   background:         $backgrounds;
 }
 
+@mixin background-with-css2-fallback(
+  $background-1,
+  $background-2: false,
+  $background-3: false,
+  $background-4: false,
+  $background-5: false,
+  $background-6: false,
+  $background-7: false,
+  $background-8: false,
+  $background-9: false,
+  $background-10: false
+) {
+  $backgrounds: compact($background-1, $background-2, $background-3, $background-4, $background-5,
+                        $background-6, $background-7, $background-8, $background-9, $background-10);
+  $mult-bgs: -compass-list-size($backgrounds) > 1;
+  $simple-background: if($mult-bgs or prefixed(-css2, $backgrounds), -css2(-compass-nth($backgrounds, last)), false);
+  @if not blank($simple-background) { background: $simple-background; }
+  @include background($background-1, $background-2, $background-3, $background-4, $background-5,
+                      $background-6, $background-7, $background-8, $background-9, $background-10);
+}
+
+
 // Background image property support for vendor prefixing within values.
 @mixin background-image(
   $image-1,
@@ -43,7 +64,8 @@
   @if $experimental-support-for-svg     and prefixed(-svg,    $images) { background-image:    -svg($images); background-size: 100%; }
   @if $experimental-support-for-webkit  and prefixed(-webkit, $images) { background-image: -webkit($images); }
   @if $experimental-support-for-mozilla and prefixed(-moz,    $images) { background-image:    -moz($images); }
-  @if $experimental-support-for-pie     and (prefixed(-pie,    $images) or -compass-list-size($images) > 1) { @warn "PIE does not support background-image. Use @include background(#{$images}) instead." }
+  @if $experimental-support-for-opera   and prefixed(-o,      $images) { background-image:      -o($images); }
+  @if $experimental-support-for-pie     and (prefixed(-pie,   $images) or -compass-list-size($images) > 1) { @warn "PIE does not support background-image. Use @include background(#{$images}) instead." }
                                                                          background-image:         $images ;
 }
 
@@ -67,6 +89,7 @@
 @mixin border-image($value) {
   @if $experimental-support-for-mozilla {    -moz-border-image:    -moz(-compass-list($value)); }
   @if $experimental-support-for-webkit  { -webkit-border-image: -webkit(-compass-list($value)); }
+  @if $experimental-support-for-opera   {      -o-border-image:      -o(-compass-list($value)); }
   @if $experimental-support-for-svg     {         border-image:    -svg(-compass-list($value)); }
                                                   border-image:                       $value;
 }
@@ -75,6 +98,7 @@
 @mixin list-style-image($image) {
   @if $experimental-support-for-mozilla and prefixed(-moz,    $image) { list-style-image:    -moz($image); }
   @if $experimental-support-for-webkit  and prefixed(-webkit, $image) { list-style-image: -webkit($image); }
+  @if $experimental-support-for-opera   and prefixed(-o,      $image) { list-style-image:      -o($image); }
   @if $experimental-support-for-svg     and prefixed(-svg,    $image) { list-style-image:    -svg($image); }
                                                                         list-style-image:         $image ;
 }
@@ -84,6 +108,7 @@
   $value: -compass-list($value);
   @if $experimental-support-for-mozilla and prefixed(-moz,    $value) { list-style-image:    -moz($value); }
   @if $experimental-support-for-webkit  and prefixed(-webkit, $value) { list-style-image: -webkit($value); }
+  @if $experimental-support-for-opera   and prefixed(-o,      $value) { list-style-image:      -o($value); }
   @if $experimental-support-for-svg     and prefixed(-svg,    $value) { list-style-image:    -svg($value); }
                                                                         list-style-image:         $value ;
 }
@@ -93,6 +118,7 @@
   $value: -compass-list($value);
   @if $experimental-support-for-mozilla and prefixed(-moz,    $value) { content:    -moz($value); }
   @if $experimental-support-for-webkit  and prefixed(-webkit, $value) { content: -webkit($value); }
+  @if $experimental-support-for-opera   and prefixed(-o,      $value) { content:      -o($value); }
   @if $experimental-support-for-svg     and prefixed(-svg,    $value) { content:    -svg($value); }
                                                                         content:         $value ;
 }

data/frameworks/compass/stylesheets/compass/utilities/sprites/_base.scss

@@ -1,3 +1,6 @@
+// Determines those states for which you want to enable magic sprite selectors
+$sprite-selectors: hover, target, active !default;
+
 // Set the width and height of an element to the original
 // dimensions of an image before it was included in the sprite.
 @mixin sprite-dimensions($map, $sprite) {
@@ -22,16 +25,17 @@
   @if $dimensions {
     @include sprite-dimensions($map, $sprite);
   }
+  @include sprite-selectors($map, $sprite, $sprite);
 }
 
 // Include the selectors for the `$sprite` given the `$map` and the 
 // `$full-sprite-name`
 // @private
-@mixin sprite_selectors($map, $sprite-name, $full-sprite-name) {
+@mixin sprite-selectors($map, $sprite-name, $full-sprite-name) {
   @each $selector in $sprite-selectors {
     @if sprite_has_selector($map, $sprite-name, $selector) {
-      .#{$full-sprite-name}:#{$selector}, .#{$full-sprite-name}_#{$selector}, .#{$full-sprite-name}-#{$selector} {
-        @include sprite($map, "#{$sprite-name}_#{$selector}");
+      &:#{$selector}, &.#{$full-sprite-name}_#{$selector}, &.#{$full-sprite-name}-#{$selector} {
+          @include sprite-background-position($map, "#{$sprite-name}_#{$selector}");
       }
     }
   }
@@ -51,7 +55,6 @@
         @if $base-class { @extend #{$base-class}; }
         @include sprite($map, $sprite-name, $dimensions);
       }
-      @include sprite_selectors($map, $sprite-name, $full-sprite-name);
     }
   }
 }

data/frameworks/compass/stylesheets/compass/utilities/sprites/_sprite-img.scss

@@ -24,8 +24,6 @@ $sprite-image-default-width: $sprite-default-size !default;
 
 $sprite-image-default-height: $sprite-default-size !default;
 
-$sprite-selectors: hover, target, active !default;
-
 // Sets all the rules for a sprite from a given sprite image to show just one of the sprites.
 // To reduce duplication use a sprite-bg mixin for common properties and a sprite-select mixin for positioning.
 @mixin sprite-img($img, $col, $row: 1, $width: $sprite-image-default-width, $height: $sprite-image-default-height, $margin: $sprite-default-margin) {

data/lib/compass/app_integration/rails.rb

@@ -11,6 +11,14 @@ module Compass
 
       extend self
 
+      def booted!
+        Compass::AppIntegration::Rails.const_set(:BOOTED, true)
+      end
+
+      def booted?
+        defined?(Compass::AppIntegration::Rails::BOOTED) && Compass::AppIntegration::Rails::BOOTED
+      end
+
       def installer(*args)
         Installer.new(*args)
       end
@@ -21,13 +29,19 @@ module Compass
       end
 
       def env
-        if rails_env = (defined?(::Rails) ? ::Rails.env : (defined?(RAILS_ENV) ? RAILS_ENV : nil))
-          rails_env.production? ? :production : :development
+        env_production? ? :production : :development
+      end
+
+      def env_production?
+        if defined?(::Rails) && ::Rails.respond_to?(:env)
+          ::Rails.env.production?
+        elsif defined?(RAILS_ENV)
+          RAILS_ENV == "production"
         end
       end
 
       def root
-        if defined?(::Rails)
+        if defined?(::Rails) && ::Rails.respond_to?(:root)
           ::Rails.root
         elsif defined?(RAILS_ROOT)
           RAILS_ROOT
@@ -35,6 +49,11 @@ module Compass
       end
 
       def initialize!(config = nil)
+        if booted?
+          Compass::Util.compass_warn("Warning: Compass was booted twice. Compass has a Railtie now; please remove your intializer.")
+        else
+          booted!
+        end
         config ||= Compass.detect_configuration_file(root)
         Compass.add_project_configuration(config, :project_type => :rails)
         Compass.discover_extensions!

data/lib/compass/commands/sprite.rb

@@ -38,14 +38,11 @@ module Compass
       end
 
       def perform
-        sprites = Compass::Sprites.new
         relative_uri = options[:uri].gsub(/^#{Compass.configuration.images_dir}\//, '')
-        sprite_images = Compass::Sprites.discover_sprites(relative_uri)
-        image_names = sprite_images.map{|i| File.basename(i, '.png')}
-        sprites.path, sprites.name = Compass::Sprites.path_and_name(relative_uri)
+        sprites = Compass::SpriteMap.new(relative_uri, Compass.sass_engine_options)
         options[:output_file] ||= File.join(Compass.configuration.sass_path, "sprites", "_#{sprites.name}.#{Compass.configuration.preferred_syntax}")
         options[:skip_overrides] ||= false
-        contents = sprites.content_for_images(relative_uri, sprites.name, image_names, options[:skip_overrides])
+        contents = sprites.content_for_images(options[:skip_overrides])
         if options[:output_file][-4..-1] != "scss"
           contents = Sass::Engine.new(contents, Compass.sass_engine_options.merge(:syntax => :scss)).to_tree.to_sass
         end

data/lib/compass/compiler.rb

@@ -7,7 +7,7 @@ module Compass
 
     def initialize(working_path, from, to, options)
       self.working_path = working_path
-      self.from, self.to = from, to
+      self.from, self.to = from.gsub('./', ''), to
       self.logger = options.delete(:logger)
       self.options = options
       self.options[:cache_location] ||= determine_cache_location
@@ -26,11 +26,15 @@ module Compass
     end
 
     def relative_stylesheet_name(sass_file)
-      sass_file[("#{from}/".length)..-1]
+      sass_file[(from.length + 1)..-1]
     end
 
     def stylesheet_name(sass_file)
-      sass_file[("#{from}/".length)..-6]
+      if sass_file.index(from) == 0
+        sass_file[(from.length + 1)..-6]
+      else
+        raise Compass::Error, "You must compile individual stylesheets from the project directory."
+      end
     end
 
     def css_files

data/lib/compass/configuration/helpers.rb

@@ -1,5 +1,6 @@
 module Compass
   module Configuration
+    @callbacks_loaded = false
     # The helpers are available as methods on the Compass module. E.g. Compass.configuration
     module Helpers
       def configuration
@@ -65,14 +66,14 @@ module Compass
             Sass::Plugin.add_template_location sass_dir, css_dir
           end
         end
-        unless defined?(CallbacksSetup)
+        unless @callbacks_loaded
           Sass::Plugin.on_updating_stylesheet do |sass_file, css_file|
             Compass.configuration.run_callback(:stylesheet_saved, css_file)
           end
           Sass::Plugin.on_compilation_error do |e, filename, css|
             Compass.configuration.run_callback(:stylesheet_error, filename, e.message)
           end
-          const_set('CallbacksSetup', true)
+          @callbacks_loaded = true
         end
       end
 

data/lib/compass/sass_extensions/functions/constants.rb

@@ -1,5 +1,12 @@
 module Compass::SassExtensions::Functions::Constants
   if defined?(Sass::Script::List)
+    POSITIONS = /top|bottom|left|right|center/
+    def is_position(position)
+      Sass::Script::Bool.new(position.is_a?(Sass::Script::String) && !!(position.value =~ POSITIONS))
+    end
+    def is_position_list(position_list)
+      Sass::Script::Bool.new(position_list.is_a?(Sass::Script::List) && position_list.value.all?{|p| is_position(p).to_bool})
+    end
     # returns the opposite position of a side or corner.
     def opposite_position(position)
       position = unless position.is_a?(Sass::Script::List)

data/lib/compass/sass_extensions/functions/cross_browser_support.rb

@@ -1,4 +1,31 @@
 module Compass::SassExtensions::Functions::CrossBrowserSupport
+
+  class CSS2FallbackValue < Sass::Script::Literal
+    attr_accessor :value, :css2_value
+    def children
+      [value, css2_value]
+    end
+    def initialize(value, css2_value)
+      self.value = value
+      self.css2_value = css2_value
+    end
+    def inspect
+      to_s
+    end
+    def to_s(options = self.options)
+      value.to_s(options)
+    end
+    def supports?(aspect)
+      aspect == "css2"
+    end
+    def has_aspect?
+      true
+    end
+    def to_css2(options = self.options)
+      css2_value
+    end
+  end
+
   # Check if any of the arguments passed require a vendor prefix.
   def prefixed(prefix, *args)
     aspect = prefix.value.sub(/^-/,"")
@@ -36,4 +63,8 @@ module Compass::SassExtensions::Functions::CrossBrowserSupport
     end
   end
 
+  def css2_fallback(value, css2_value)
+    CSS2FallbackValue.new(value, css2_value)
+  end
+
 end

data/lib/compass/sass_extensions/functions/gradient_support.rb

@@ -1,6 +1,6 @@
 module Compass::SassExtensions::Functions::GradientSupport
 
-  GRADIENT_ASPECTS = %w(webkit moz svg pie css2).freeze
+  GRADIENT_ASPECTS = %w(webkit moz svg pie css2 o).freeze
 
   class ColorStop < Sass::Script::Literal
     attr_accessor :color, :stop
@@ -34,15 +34,15 @@ module Compass::SassExtensions::Functions::GradientSupport
   end
 
   class RadialGradient < Sass::Script::Literal
-    attr_accessor :position_and_angle, :shape_and_size, :color_stops
+    attr_accessor :position_or_angle, :shape_and_size, :color_stops
     def children
-      [color_stops, position_and_angle, shape_and_size].compact
+      [color_stops, position_or_angle, shape_and_size].compact
     end
-    def initialize(position_and_angle, shape_and_size, color_stops)
+    def initialize(position_or_angle, shape_and_size, color_stops)
       unless color_stops.value.size >= 2
         raise Sass::SyntaxError, "At least two color stops are required for a radial-gradient"
       end
-      self.position_and_angle = position_and_angle
+      self.position_or_angle = position_or_angle
       self.shape_and_size = shape_and_size
       self.color_stops = color_stops
     end
@@ -51,22 +51,31 @@ module Compass::SassExtensions::Functions::GradientSupport
     end
     def to_s(options = self.options)
       s = "radial-gradient("
-      s << position_and_angle.to_s(options) << ", " if position_and_angle
+      s << position_or_angle.to_s(options) << ", " if position_or_angle
       s << shape_and_size.to_s(options) << ", " if shape_and_size
       s << color_stops.to_s(options)
       s << ")"
     end
     def supports?(aspect)
-      GRADIENT_ASPECTS.include?(aspect)
+      if aspect == "svg"
+        if position_or_angle.nil? || is_position(position_or_angle).to_bool || is_position_list(position_or_angle).to_bool
+          true
+        else
+          Compass::Util.compass_warn("Warning: Angle-based gradients are not yet supported in SVG. Found: #{position_or_angle}")
+          false
+        end
+      else
+        GRADIENT_ASPECTS.include?(aspect)
+      end
     end
     def has_aspect?
       true
     end
     def to_webkit(options = self.options)
       args = [
-        grad_point(position_and_angle || _center_position),
+        grad_point(position_or_angle || _center_position),
         Sass::Script::String.new("0"),
-        grad_point(position_and_angle || _center_position),
+        grad_point(position_or_angle || _center_position),
         grad_end_position(color_stops, Sass::Script::Bool.new(true)),
         grad_color_stops(color_stops)
       ]
@@ -77,9 +86,12 @@ module Compass::SassExtensions::Functions::GradientSupport
     def to_moz(options = self.options)
       Sass::Script::String.new("-moz-#{to_s(options)}")
     end
+    def to_o(options = self.options)
+      Sass::Script::String.new("-o-#{to_s(options)}")
+    end
     def to_svg(options = self.options)
       # XXX Add shape support if possible
-      radial_svg_gradient(color_stops, position_and_angle || _center_position)
+      radial_svg_gradient(color_stops, position_or_angle || _center_position)
     end
     def to_pie(options = self.options)
       Compass::Logger.new.record(:warning, "PIE does not support radial-gradient.")
@@ -91,15 +103,15 @@ module Compass::SassExtensions::Functions::GradientSupport
   end
 
   class LinearGradient < Sass::Script::Literal
-    attr_accessor :color_stops, :position_and_angle
+    attr_accessor :color_stops, :position_or_angle
     def children
-      [color_stops, position_and_angle].compact
+      [color_stops, position_or_angle].compact
     end
-    def initialize(position_and_angle, color_stops)
+    def initialize(position_or_angle, color_stops)
       unless color_stops.value.size >= 2
         raise Sass::SyntaxError, "At least two color stops are required for a linear-gradient"
       end
-      self.position_and_angle = position_and_angle
+      self.position_or_angle = position_or_angle
       self.color_stops = color_stops
     end
     def inspect
@@ -107,20 +119,29 @@ module Compass::SassExtensions::Functions::GradientSupport
     end
     def to_s(options = self.options)
       s = "linear-gradient("
-      s << position_and_angle.to_s(options) << ", " if position_and_angle
+      s << position_or_angle.to_s(options) << ", " if position_or_angle
       s << color_stops.to_s(options)
       s << ")"
     end
     def supports?(aspect)
-      GRADIENT_ASPECTS.include?(aspect)
+      if aspect == "svg"
+        if position_or_angle.nil? || is_position(position_or_angle).to_bool || is_position_list(position_or_angle).to_bool
+          true
+        else
+          Compass::Util.compass_warn("Warning: Angle-based gradients are not yet supported in SVG. Found: #{position_or_angle}")
+          false
+        end
+      else
+        GRADIENT_ASPECTS.include?(aspect)
+      end
     end
     def has_aspect?
       true
     end
     def to_webkit(options = self.options)
       args = []
-      args << grad_point(position_and_angle || Sass::Script::String.new("top"))
-      args << grad_point(opposite_position(position_and_angle || Sass::Script::String.new("top")))
+      args << grad_point(position_or_angle || Sass::Script::String.new("top"))
+      args << grad_point(opposite_position(position_or_angle || Sass::Script::String.new("top")))
       args << grad_color_stops(color_stops)
       args.each{|a| a.options = options}
       Sass::Script::String.new("-webkit-gradient(linear, #{args.join(', ')})")
@@ -128,8 +149,11 @@ module Compass::SassExtensions::Functions::GradientSupport
     def to_moz(options = self.options)
       Sass::Script::String.new("-moz-#{to_s(options)}")
     end
+    def to_o(options = self.options)
+      Sass::Script::String.new("-o-#{to_s(options)}")
+    end
     def to_svg(options = self.options)
-      linear_svg_gradient(color_stops, position_and_angle || Sass::Script::String.new("top"))
+      linear_svg_gradient(color_stops, position_or_angle || Sass::Script::String.new("top"))
     end
     def to_pie(options = self.options)
       # PIE just uses the standard rep, but the property is prefixed so
@@ -144,13 +168,18 @@ module Compass::SassExtensions::Functions::GradientSupport
   module Functions
 
     # given a position list, return a corresponding position in percents
+    # otherwise, returns the original argument
     def grad_point(position)
+      original_value = position
       position = unless position.is_a?(Sass::Script::List)
         Sass::Script::List.new([position], :space)
       else
         Sass::Script::List.new(position.value.dup, position.separator)
       end
-      position.value.reject!{|p| p.is_a?(Sass::Script::Number) && p.numerator_units.include?("deg")}
+      # Handle unknown arguments by passing them along untouched.
+      unless position.value.all?{|p| is_position(p).to_bool }
+        return original_value
+      end
       if (position.value.first.value =~ /top|bottom/) or (position.value.last.value =~ /left|right/)
         # browsers are pretty forgiving of reversed positions so we are too.
         position.value.reverse!
@@ -192,7 +221,7 @@ module Compass::SassExtensions::Functions::GradientSupport
       end, :comma)
     end
 
-    def radial_gradient(position_and_angle, shape_and_size, *color_stops)
+    def radial_gradient(position_or_angle, shape_and_size, *color_stops)
       # Have to deal with variable length/meaning arguments.
       if color_stop?(shape_and_size)
         color_stops.unshift(shape_and_size)
@@ -203,38 +232,38 @@ module Compass::SassExtensions::Functions::GradientSupport
         shape_and_size = nil
       end
       shape_and_size = nil if shape_and_size && !shape_and_size.to_bool # nil out explictly passed falses
-      # ditto for position_and_angle
-      if color_stop?(position_and_angle)
-        color_stops.unshift(position_and_angle)
-        position_and_angle = nil
-      elsif list_of_color_stops?(position_and_angle)
-        color_stops = position_and_angle.value + color_stops
-        position_and_angle = nil
+      # ditto for position_or_angle
+      if color_stop?(position_or_angle)
+        color_stops.unshift(position_or_angle)
+        position_or_angle = nil
+      elsif list_of_color_stops?(position_or_angle)
+        color_stops = position_or_angle.value + color_stops
+        position_or_angle = nil
       end
-      position_and_angle = nil if position_and_angle && !position_and_angle.to_bool
+      position_or_angle = nil if position_or_angle && !position_or_angle.to_bool
 
       # Support legacy use of the color-stops() function
       if color_stops.size == 1 && list_of_color_stops?(color_stops.first)
         color_stops = color_stops.first.value
       end
-      RadialGradient.new(position_and_angle, shape_and_size, send(:color_stops, *color_stops))
+      RadialGradient.new(position_or_angle, shape_and_size, send(:color_stops, *color_stops))
     end
 
-    def linear_gradient(position_and_angle, *color_stops)
-      if color_stop?(position_and_angle)
-        color_stops.unshift(position_and_angle)
-        position_and_angle = nil
-      elsif list_of_color_stops?(position_and_angle)
-        color_stops = position_and_angle.value + color_stops
-        position_and_angle = nil
+    def linear_gradient(position_or_angle, *color_stops)
+      if color_stop?(position_or_angle)
+        color_stops.unshift(position_or_angle)
+        position_or_angle = nil
+      elsif list_of_color_stops?(position_or_angle)
+        color_stops = position_or_angle.value + color_stops
+        position_or_angle = nil
       end
-      position_and_angle = nil if position_and_angle && !position_and_angle.to_bool
+      position_or_angle = nil if position_or_angle && !position_or_angle.to_bool
 
       # Support legacy use of the color-stops() function
-      if color_stops.size == 1 && list_of_color_stops?(color_stops.first)
-        color_stops = color_stops.first.value
+      if color_stops.size == 1 && (stops = list_of_color_stops?(color_stops.first))
+        color_stops = stops
       end
-      LinearGradient.new(position_and_angle, send(:color_stops, *color_stops))
+      LinearGradient.new(position_or_angle, send(:color_stops, *color_stops))
     end
 
     # returns color-stop() calls for use in webkit.
@@ -255,8 +284,8 @@ module Compass::SassExtensions::Functions::GradientSupport
         stop = pos.stop
         stop = stop.div(max).times(Sass::Script::Number.new(100,["%"])) if stop.numerator_units == max.numerator_units && max.numerator_units != ["%"]
         # Make sure the color stops are specified in the right order.
-        if last_value && last_value.value > stop.value
-          raise Sass::SyntaxError.new("Color stops must be specified in increasing order")
+        if last_value && stop.numerator_units == last_value.numerator_units && stop.denominator_units == last_value.denominator_units && stop.value < last_value.value
+          raise Sass::SyntaxError.new("Color stops must be specified in increasing order. #{stop.value} came after #{last_value.value}.")
         end
         last_value = stop
         [stop, pos.color]
@@ -386,7 +415,11 @@ module Compass::SassExtensions::Functions::GradientSupport
     end
 
     def list_of_color_stops?(arg)
-      arg.value.is_a?(Array) && arg.value.all?{|a| a.is_a?(ColorStop)}
+      if arg.respond_to?(:value)
+        arg.value.is_a?(Array) && arg.value.all?{|a| color_stop?(a)} ? arg.value : nil
+      elsif arg.is_a?(Array)
+        arg.all?{|a| color_stop?(a)} ? arg : nil
+      end
     end
 
     def linear_svg(color_stops, x1, y1, x2, y2)