glimmer-dsl-swt 4.18.4.6 → 4.18.4.11

data/glimmer-dsl-swt.gemspec

@@ -2,22 +2,28 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: glimmer-dsl-swt 4.18.4.6 ruby lib
+# stub: glimmer-dsl-swt 4.18.4.11 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "glimmer-dsl-swt".freeze
-  s.version = "4.18.4.6"
+  s.version = "4.18.4.11"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["AndyMaleh".freeze]
-  s.date = "2021-02-11"
+  s.date = "2021-02-19"
   s.description = "Glimmer DSL for SWT (JRuby Desktop Development GUI Framework) is a native-GUI cross-platform desktop development library written in JRuby, an OS-threaded faster JVM version of Ruby. Glimmer's main innovation is a declarative Ruby DSL that enables productive and efficient authoring of desktop application user-interfaces by relying on the robust Eclipse SWT library. Glimmer additionally innovates by having built-in data-binding support, which greatly facilitates synchronizing the GUI with domain models, thus achieving true decoupling of object oriented components and enabling developers to solve business problems (test-first) without worrying about GUI concerns, or alternatively drive development GUI-first, and then write clean business models (test-first) afterwards. Not only does Glimmer provide a large set of GUI widgets, but it also supports drawing Canvas Graphics like Shapes and Animations. To get started quickly, Glimmer offers scaffolding options for Apps, Gems, and Custom Widgets. Glimmer also includes native-executable packaging support, sorely lacking in other libraries, thus enabling the delivery of desktop apps written in Ruby as truly native DMG/PKG/APP files on the Mac + App Store, MSI/EXE files on Windows, and Gem Packaged Shell Scripts on Linux.".freeze
   s.email = "andy.am@gmail.com".freeze
   s.executables = ["glimmer".freeze, "girb".freeze]
   s.extra_rdoc_files = [
-    "LICENSE.txt",
-    "README.md"
+    "README.md",
+    "docs/reference/GLIMMER_COMMAND.md",
+    "docs/reference/GLIMMER_CONFIGURATION.md",
+    "docs/reference/GLIMMER_GIRB.md",
+    "docs/reference/GLIMMER_GUI_DSL_SYNTAX.md",
+    "docs/reference/GLIMMER_PACKAGING_AND_DISTRIBUTION.md",
+    "docs/reference/GLIMMER_SAMPLES.md",
+    "docs/reference/GLIMMER_STYLE_GUIDE.md"
   ]
   s.files = [
     "CHANGELOG.md",
@@ -171,6 +177,7 @@ Gem::Specification.new do |s|
     "samples/hello/hello_combo.rb",
     "samples/hello/hello_computed.rb",
     "samples/hello/hello_computed/contact.rb",
+    "samples/hello/hello_cursor.rb",
     "samples/hello/hello_custom_shell.rb",
     "samples/hello/hello_custom_widget.rb",
     "samples/hello/hello_date_time.rb",
@@ -186,6 +193,7 @@ Gem::Specification.new do |s|
     "samples/hello/hello_menu_bar.rb",
     "samples/hello/hello_message_box.rb",
     "samples/hello/hello_pop_up_context_menu.rb",
+    "samples/hello/hello_progress_bar.rb",
     "samples/hello/hello_radio.rb",
     "samples/hello/hello_radio_group.rb",
     "samples/hello/hello_sash_form.rb",

data/lib/glimmer/data_binding/widget_binding.rb

@@ -32,10 +32,12 @@ module Glimmer
       include Observer
 
       attr_reader :widget, :property
-      def initialize(widget, property, translator = nil)
+      def initialize(widget, property, translator = nil, sync_exec: false, async_exec: false)
         @widget = widget
         @property = property
         @translator = translator || proc {|value| value} #TODO check on this it doesn't seem used
+        @sync_exec = sync_exec
+        @async_exec = async_exec
 
         if @widget.respond_to?(:on_widget_disposed)
           @widget.on_widget_disposed do |dispose_event|
@@ -54,8 +56,10 @@ module Glimmer
           end
           @widget.set_attribute(@property, converted_value) unless evaluate_property == converted_value
         end
-        if Config.auto_sync_exec? && Config.require_sync_exec?
+        if @sync_exec || Config.auto_sync_exec? && Config.require_sync_exec?
           SWT::DisplayProxy.instance.sync_exec(&update_operation)
+        elsif @async_exec
+          SWT::DisplayProxy.instance.async_exec(&update_operation)
         else
           update_operation.call
         end
@@ -66,7 +70,16 @@ module Glimmer
           unregister_all_observables
           return
         end
-        @widget.get_attribute(@property)
+        read_operation = lambda do
+          @widget.get_attribute(@property)
+        end
+        if @sync_exec || Config.auto_sync_exec? && Config.require_sync_exec?
+          SWT::DisplayProxy.instance.sync_exec(&read_operation)
+        elsif @async_exec
+          SWT::DisplayProxy.instance.async_exec(&read_operation)
+        else
+          read_operation.call
+        end
       end
     end
   end

data/lib/glimmer/dsl/swt/color_expression.rb

@@ -32,7 +32,7 @@ module Glimmer
         include_package 'org.eclipse.swt.widgets'
   
         def interpret(parent, keyword, *args, &block)
-          Glimmer::SWT::ColorProxy.flyweight(*args)
+          Glimmer::SWT::ColorProxy.create(*args)
         end
       end
     end

data/lib/glimmer/dsl/swt/custom_widget_expression.rb

@@ -46,11 +46,14 @@ module Glimmer
   
         def interpret(parent, keyword, *args, &block)
           options = args.last.is_a?(Hash) ? args.pop : {}
-          UI::CustomWidget.for(keyword).new(parent, *args, options, &block)
+          UI::CustomWidget.for(keyword).new(parent, *args, options, &block).tap do |new_custom_widget|
+            new_custom_widget.body_root.paint_pixel_by_pixel(&block) if block&.parameters&.count == 2
+          end
         end
   
         def add_content(parent, &block)
           # TODO consider avoiding source_location
+          return if block&.parameters&.count == 2
           if block.source_location == parent.content&.__getobj__.source_location
             parent.content.call(parent) unless parent.content.called?
           else

data/lib/glimmer/dsl/swt/data_binding_expression.rb

@@ -22,6 +22,7 @@
 require 'glimmer/dsl/expression'
 require 'glimmer/data_binding/model_binding'
 require 'glimmer/data_binding/widget_binding'
+require 'glimmer/swt/display_proxy'
 
 module Glimmer
   module DSL
@@ -41,7 +42,7 @@ module Glimmer
   
         def interpret(parent, keyword, *args, &block)
           model_binding = args[0]
-          widget_binding_parameters = [parent, keyword]
+          widget_binding_parameters = [parent, keyword, {async_exec: model_binding.binding_options[:async_exec], sync_exec: model_binding.binding_options[:sync_exec]}]
           widget_binding = DataBinding::WidgetBinding.new(*widget_binding_parameters)
           widget_binding.call(model_binding.evaluate_property)
           #TODO make this options observer dependent and all similar observers in widget specific data binding handlers

data/lib/glimmer/dsl/swt/image_expression.rb

@@ -20,7 +20,9 @@
 # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 require 'glimmer/dsl/expression'
+require 'glimmer/dsl/top_level_expression'
 require 'glimmer/swt/image_proxy'
+require 'glimmer/swt/widget_proxy'
 
 module Glimmer
   module DSL
@@ -28,19 +30,28 @@ module Glimmer
       # image expression
       # Note: Cannot be a static expression because it clashes with image property expression
       class ImageExpression < Expression
+        include TopLevelExpression
         include ParentExpression
         
         def can_interpret?(parent, keyword, *args, &block)
           (keyword == 'image') and
-            (parent.nil? or parent.respond_to?('image='))
+            (parent.nil? or parent.respond_to?('image=') or args.first.is_a?(Numeric))
         end
   
         def interpret(parent, keyword, *args, &block)
-          args.unshift(parent) unless parent.nil?
-          Glimmer::SWT::ImageProxy.new(*args, &block)
+          options = args.last.is_a?(Hash) ? args.last : {}
+          coordinate_args = args.size == (options.empty? ? 2 : 3)
+          args.unshift(parent) unless parent.nil? || options[:top_level]
+          @create_pixel_by_pixel = coordinate_args && block&.parameters&.count == 2
+          if @create_pixel_by_pixel
+            Glimmer::SWT::ImageProxy.create_pixel_by_pixel(*args, &block)
+          else
+            Glimmer::SWT::ImageProxy.create(*args, &block)
+          end
         end
 
         def add_content(parent, &block)
+          return if @create_pixel_by_pixel || block&.parameters&.count == 2
           super
           parent.post_add_content
         end

data/lib/glimmer/dsl/swt/radio_group_selection_data_binding_expression.rb

@@ -51,6 +51,7 @@ module Glimmer
           widget_binding.call(model_binding.evaluate_property)
           widget_binding.observe(model, model_binding.property_name_expression)
   
+          raise(Glimmer::Error, "No radios found! Make sure radio selection is data-bound to a property having property_options as non-empty array!") if parent.items.empty?
           parent.on_widget_selected do
             model_binding.call(widget_binding.evaluate_property)
           end

data/lib/glimmer/dsl/swt/shell_expression.rb

@@ -32,7 +32,7 @@ module Glimmer
         include ParentExpression
 
         def can_interpret?(parent, keyword, *args, &block)
-          keyword == 'shell' and
+          super and
             (parent.nil? or parent.is_a?(Glimmer::SWT::ShellProxy))
         end
   
@@ -41,6 +41,9 @@ module Glimmer
           Glimmer::SWT::ShellProxy.send(:new, *args)
         end
       end
+      class WindowExpression < ShellExpression
+        # Alias
+      end
     end
   end
 end

data/lib/glimmer/dsl/swt/widget_expression.rb

@@ -40,10 +40,13 @@ module Glimmer
         end
 
         def interpret(parent, keyword, *args, &block)
-          Glimmer::SWT::WidgetProxy.create(keyword, parent, args)
+          Glimmer::SWT::WidgetProxy.create(keyword, parent, args).tap do |new_widget_proxy|
+            new_widget_proxy.paint_pixel_by_pixel(&block) if block&.parameters&.count == 2
+          end
         end
 
         def add_content(parent, &block)
+          return if block&.parameters&.count == 2
           super
           parent.post_add_content
           parent.finish_add_content!

data/lib/glimmer/swt/color_proxy.rb

@@ -33,7 +33,7 @@ module Glimmer
       include_package 'org.eclipse.swt.graphics'
       
       class << self
-        def flyweight(*args)
+        def create(*args)
           flyweight_color_proxies[args] ||= new(*args)
         end
         
@@ -59,8 +59,10 @@ module Glimmer
       # rgba is 4 arguments representing Red, Green, Blue, and Alpha numeric values
       #
       def initialize(*args)
+        @options = args.last.is_a?(Hash) ? args.pop : {}
         @args = args
-        ensure_arg_values_within_valid_bounds
+        @args = @args.first if @args.first.is_a?(Array)
+        ensure_arg_values_within_valid_bounds unless @options[:ensure_bounds] == false
       end
 
       def swt_color
@@ -75,8 +77,7 @@ module Glimmer
               @swt_color = @args.first
             end
           when 3..4
-            red, green, blue, alpha = @args
-            @swt_color = Color.new(*[red, green, blue, alpha].compact)
+            @swt_color = Color.new(*@args)
           end
         end
         @swt_color

data/lib/glimmer/swt/custom/animation.rb

@@ -84,6 +84,7 @@ module Glimmer
         
         def initialize(parent)
           @parent = parent
+          @parent.requires_shape_disposal = true
           @started = true
           @frame_index = 0
           @cycle_count_index = 0

data/lib/glimmer/swt/custom/code_text.rb

@@ -64,6 +64,18 @@ module Glimmer
           respond_to?(method_name)
         end
         
+        def can_handle_observation_request?(observation_request)
+          @styled_text_proxy.can_handle_observation_request?(observation_request)
+        rescue
+          super
+        end
+        
+        def handle_observation_request(observation_request, &block)
+          @styled_text_proxy.handle_observation_request(observation_request, &block)
+        rescue
+          super
+        end
+        
         def root_block=(block)
           body_root.content(&block)
         end
@@ -153,6 +165,7 @@ module Glimmer
             top_margin 5
             right_margin 5
             bottom_margin 5
+            tabs 2
             
             if default_behavior
               on_key_pressed { |event|
@@ -166,6 +179,15 @@ module Glimmer
                   jump_to_end_of_line
                 end
               }
+              on_verify_text { |verify_event|
+                if verify_event.text == "\n"
+                  line_index = verify_event.widget.get_line_at_offset(verify_event.widget.get_caret_offset)
+                  line = verify_event.widget.get_line(line_index)
+                  line_indent = line.match(/^([ ]*)/)[1].to_s.size
+                  verify_event.text += ' '*line_indent
+                  verify_event.text += ' '*2 if line.strip.end_with?('{') || line.strip.match(/do([ ]*[|][^|]*[|])?$/) || line.start_with?('class') || line.start_with?('module') || line.strip.start_with?('def')
+                end
+              }
             end
             
             on_modify_text { |event|

data/lib/glimmer/swt/custom/drawable.rb

@@ -24,12 +24,75 @@ module Glimmer
     module Custom
       # Represents SWT drawable controls (widgets like canvas) and display
       module Drawable
+        attr_accessor :requires_shape_disposal, :image_double_buffered
+        alias requires_shape_disposal? requires_shape_disposal
+        alias image_double_buffered? image_double_buffered
+        
+        include_package 'org.eclipse.swt.graphics'
+      
         def shapes
           @shapes ||= []
         end
         
-        def clear_shapes
-          shapes.dup.each(&:dispose)
+        def image_buffered_shapes
+          @image_buffered_shapes ||= []
+        end
+        
+        def add_shape(shape)
+          if !@image_double_buffered || shape.args.first == @image_proxy_buffer
+            shapes << shape
+          else
+            image_buffered_shapes << shape
+          end
+        end
+        
+        def clear_shapes(dispose_images: true, dispose_patterns: true)
+          # Optimize further by having a collection of disposable_shapes independent of shapes, which is much smaller and only has shapes that require disposal (shapes with patterns or image)
+          shapes.dup.each {|s| s.dispose(dispose_images: dispose_images, dispose_patterns: dispose_patterns) } if requires_shape_disposal?
+        end
+        
+        def paint_pixel_by_pixel(width = nil, height = nil, &each_pixel_color)
+          if @image_double_buffered
+            work = lambda do |paint_event|
+              width ||= swt_drawable.bounds.width
+              height ||= swt_drawable.bounds.height
+              @image_proxy_buffer ||= ImageProxy.create_pixel_by_pixel(width, height, &each_pixel_color)
+              @image_proxy_buffer.shape(self).paint(paint_event)
+            end
+          else
+            work = lambda do |paint_event_or_image|
+              the_gc = paint_event_or_image.gc
+              current_foreground = nil
+              width ||= swt_drawable.bounds.width
+              height ||= swt_drawable.bounds.height
+              height.times do |y|
+                width.times do |x|
+                  new_foreground = each_pixel_color.call(x, y)
+                  new_foreground = Glimmer::SWT::ColorProxy.create(new_foreground, ensure_bounds: false) unless new_foreground.is_a?(ColorProxy) || new_foreground.is_a?(Color)
+                  new_foreground = new_foreground.swt_color if new_foreground.is_a?(Glimmer::SWT::ColorProxy)
+                  the_gc.foreground = current_foreground = new_foreground unless new_foreground == current_foreground
+                  the_gc.draw_point x, y
+                end
+              end
+            end
+          end
+          if respond_to?(:gc)
+            work.call(self)
+          else
+            on_swt_paint(&work)
+          end
+        end
+        
+        def swt_drawable
+          swt_drawable = nil
+          if respond_to?(:swt_image)
+            swt_drawable = swt_image
+          elsif respond_to?(:swt_display)
+            swt_drawable = swt_display
+          elsif respond_to?(:swt_widget)
+            swt_drawable = swt_widget
+          end
+          swt_drawable
         end
         
         def deregister_shape_painting
@@ -40,8 +103,21 @@ module Glimmer
           # TODO consider performance optimization relating to order of shape rendering (affecting only further shapes not previous ones)
           if @paint_listener_proxy.nil?
             shape_painter = lambda do |paint_event|
-              shapes.each do |shape|
-                shape.paint(paint_event)
+              shape_painting_work = lambda do |paint_event|
+                paintable_shapes = @image_double_buffered ? image_buffered_shapes : shapes
+                paintable_shapes.each do |shape|
+                  shape.paint(paint_event)
+                end
+              end
+              if @image_double_buffered
+                if @image_proxy_buffer.nil?
+                  swt_image = Image.new(DisplayProxy.instance.swt_display, bounds.width, bounds.height)
+                  @image_proxy_buffer = ImageProxy.new(swt_image: swt_image)
+                  shape_painting_work.call(@image_proxy_buffer)
+                end
+                @image_proxy_buffer.shape(self).paint(paint_event)
+              else
+                shape_painting_work.call(paint_event)
               end
             end
             

data/lib/glimmer/swt/custom/radio_group.rb

@@ -73,7 +73,8 @@ module Glimmer
         end
         
         def can_handle_observation_request?(observation_request)
-          radios.first&.can_handle_observation_request?(observation_request) || super(observation_request)
+          radios.first&.can_handle_observation_request?(observation_request) or
+            super(observation_request)
         end
         
         def handle_observation_request(observation_request, &block)

data/lib/glimmer/swt/custom/shape.rb

@@ -103,7 +103,7 @@ module Glimmer
           @options.reject {|key, value| %w[fill gradient round].include?(key.to_s)}.each do |property, property_args|
             @properties[property] = property_args
           end
-          @parent.shapes << self
+          @parent.add_shape(self)
           post_add_content if property_block.nil?
         end
         
@@ -133,8 +133,8 @@ module Glimmer
         
         def post_add_content
           unless @content_added
-            amend_method_name_options_based_on_properties
-            @parent.setup_shape_painting
+            amend_method_name_options_based_on_properties!
+            @parent.setup_shape_painting unless @parent.is_a?(ImageProxy)
             @content_added = true
           end
         end
@@ -142,7 +142,10 @@ module Glimmer
         def apply_property_arg_conversions(method_name, property, args)
           args = args.dup
           the_java_method = org.eclipse.swt.graphics.GC.java_class.declared_instance_methods.detect {|m| m.name == method_name}
-          if (args.first.is_a?(Symbol) || args.first.is_a?(String))
+          if ['setBackground', 'setForeground'].include?(method_name.to_s) && args.first.is_a?(Array)
+            args[0] = ColorProxy.new(args[0])
+          end
+          if args.first.is_a?(Symbol) || args.first.is_a?(String)
             if the_java_method.parameter_types.first == Color.java_class
               args[0] = ColorProxy.new(args[0])
             end
@@ -163,6 +166,7 @@ module Glimmer
             args[0] = args[0].swt_transform
           end
           if ['setBackgroundPattern', 'setForegroundPattern'].include?(method_name.to_s)
+            @parent.requires_shape_disposal = true
             args.each_with_index do |arg, i|
               if arg.is_a?(Symbol) || arg.is_a?(String)
                 args[i] = ColorProxy.new(arg).swt_color
@@ -177,41 +181,48 @@ module Glimmer
           args
         end
         
-        def apply_shape_arg_conversions(method_name, args)
-          if args.size > 1 && (method_name.include?('polygon') || method_name.include?('polyline'))
-            args[0] = args.dup
-            args[1..-1] = []
+        def apply_shape_arg_conversions!
+          if @args.size > 1 && (['polygon', 'polyline'].include?(@name))
+            @args[0] = @args.dup
+            @args[1..-1] = []
           end
         end
         
-        def apply_shape_arg_defaults(method_name, args)
-          if method_name.include?('round_rectangle') && args.size.between?(4, 5)
-            (6 - args.size).times {args << 60}
-          elsif method_name.include?('rectangle') && gradient? && args.size == 4
-            args << true
-          elsif (method_name.include?('text') || method_name.include?('string')) && !@properties.keys.map(&:to_s).include?('background') && args.size == 3
-            args << true
-          end
-          if method_name.include?('image') && args.first.is_a?(String)
-            args[0] = ImageProxy.new(args[0])
+        def apply_shape_arg_defaults!
+          if @name.include?('rectangle') && round? && @args.size.between?(4, 5)
+            (6 - @args.size).times {@args << 60}
+          elsif @name.include?('rectangle') && gradient? && @args.size == 4
+            @args << true
+          elsif (@name.include?('text') || @name.include?('String')) && !@properties.keys.map(&:to_s).include?('background') && @args.size == 3
+            @args << true
           end
-          if method_name.include?('image') && args.first.is_a?(ImageProxy)
-            @image = args[0] = args[0].swt_image
+          if @name.include?('image')
+            @parent.requires_shape_disposal = true
+            if @args.size == 1
+              @args[1] = 0
+              @args[2] = 0
+            end
+            if @args.first.is_a?(String)
+              @args[0] = ImageProxy.new(@args[0])
+            end
+            if @args.first.is_a?(ImageProxy)
+              @image = @args[0] = @args[0].swt_image
+            end
           end
         end
                 
         # Tolerates shape extra args added by user by mistake
         # (e.g. happens when switching from round rectangle to a standard one without removing all extra args)
-        def tolerate_shape_extra_args(method_name, args)
+        def tolerate_shape_extra_args!
           the_java_method_arg_count = org.eclipse.swt.graphics.GC.java_class.declared_instance_methods.select do |m|
-            m.name == method_name.camelcase(:lower)
+            m.name == @method_name.camelcase(:lower)
           end.map(&:parameter_types).map(&:size).max
-          if args.size > the_java_method_arg_count
-            args[the_java_method_arg_count..-1] = []
+          if @args.size > the_java_method_arg_count
+            @args[the_java_method_arg_count..-1] = []
           end
         end
         
-        def amend_method_name_options_based_on_properties
+        def amend_method_name_options_based_on_properties!
           return if @name == 'point'
           if has_some_background? && !has_some_foreground?
             @options[:fill] = true
@@ -252,16 +263,20 @@ module Glimmer
           the_pattern
         end
         
-        def dispose
-          @background_pattern&.dispose
-          @background_pattern = nil
-          @foreground_pattern&.dispose
-          @foreground_pattern = nil
-          @image&.dispose
-          @image = nil
+        def dispose(dispose_images: true, dispose_patterns: true)
+          if dispose_patterns
+            @background_pattern&.dispose
+            @background_pattern = nil
+            @foreground_pattern&.dispose
+            @foreground_pattern = nil
+          end
+          if dispose_images
+            @image&.dispose
+            @image = nil
+          end
           @parent.shapes.delete(self)
         end
-        
+                
         def paint(paint_event)
           calculate_paint_args!
           @properties.each do |property, args|
@@ -276,20 +291,39 @@ module Glimmer
         end
         
         def calculate_paint_args!
-          unless @name == 'point' || @calculated_paint_args
-            @properties['background'] = [@parent.background] if fill? && !has_some_background?
-            @properties['foreground'] = [@parent.foreground] if @parent.respond_to?(:foreground) && draw? && !has_some_foreground?
-            @properties['font'] = [@parent.font] if @parent.respond_to?(:font) && draw? && !@properties.keys.map(&:to_s).include?('font')
-            @properties['transform'] = [nil] if @parent.respond_to?(:transform) && !@properties.keys.map(&:to_s).include?('transform')
-            @properties.each do |property, args|
-              method_name = attribute_setter(property)
-              converted_args = apply_property_arg_conversions(method_name, property, args)
-              @properties[property] = converted_args
+          unless @calculated_paint_args
+            if @name == 'point'
+              # optimized performance calculation for pixel points
+              if !@properties[:foreground].is_a?(Color)
+                if @properties[:foreground].is_a?(Array)
+                  @properties[:foreground] = ColorProxy.new(@properties[:foreground], ensure_bounds: false)
+                end
+                if @properties[:foreground].is_a?(Symbol) || @properties[:foreground].is_a?(String)
+                 @properties[:foreground] = ColorProxy.new(@properties[:foreground], ensure_bounds: false)
+                end
+                if @properties[:foreground].is_a?(ColorProxy)
+                  @properties[:foreground] = @properties[:foreground].swt_color
+                end
+              end
+            else
+              @properties['background'] = [@parent.background] if fill? && !has_some_background?
+              @properties['foreground'] = [@parent.foreground] if @parent.respond_to?(:foreground) && draw? && !has_some_foreground?
+              # TODO regarding alpha, make sure to reset it to parent stored alpha once we allow setting shape properties on parents directly without shapes
+              @properties['alpha'] ||= [255]
+              @properties['font'] = [@parent.font] if @parent.respond_to?(:font) && draw? && !@properties.keys.map(&:to_s).include?('font')
+              # TODO regarding transform, make sure to reset it to parent stored alpha once we allow setting shape properties on parents directly without shapes
+              # Also do that with all future-added properties
+              @properties['transform'] = [nil] if @parent.respond_to?(:transform) && !@properties.keys.map(&:to_s).include?('transform')
+              @properties.each do |property, args|
+                method_name = attribute_setter(property)
+                converted_args = apply_property_arg_conversions(method_name, property, args)
+                @properties[property] = converted_args
+              end
+              apply_shape_arg_conversions!
+              apply_shape_arg_defaults!
+              tolerate_shape_extra_args!
+              @calculated_paint_args = true
             end
-            apply_shape_arg_conversions(@method_name, @args)
-            apply_shape_arg_defaults(@method_name, @args)
-            tolerate_shape_extra_args(@method_name, @args)
-            @calculated_paint_args = true
           end
         end