glimmer-dsl-swt 4.18.4.8 → 4.18.5.1

data/lib/glimmer/swt/swt_proxy.rb

@@ -49,10 +49,27 @@ module Glimmer
         def extra_styles
           EXTRA_STYLES
         end
+        
+        def cursor_styles
+          SWT.constants.select {|c| c.to_s.start_with?('CURSOR_')}
+        end
+        
+        def color_styles
+          SWT.constants.select {|c| c.to_s.start_with?('COLOR_')}
+        end
+        
+        def cursor_options
+          cursor_styles.map {|c| c.to_s.sub('CURSOR_', '').downcase}.map(&:to_sym)
+        end
+        
+        def color_options
+          color_styles.map {|c| c.to_s.sub('COLOR_', '').downcase}.map(&:to_sym)
+        end
       end
       
       EXTRA_STYLES = {
         NO_RESIZE: self[:shell_trim, :resize!, :max!],
+        WINDOW_TRIM: self[:shell_trim],
       }
     end
   end

data/lib/glimmer/swt/tab_item_proxy.rb

@@ -49,6 +49,7 @@ module Glimmer
         @widget_proxy = SWT::WidgetProxy.new('tab_item', parent, style)
         @swt_tab_item = @widget_proxy.swt_widget
         @swt_tab_item.control = swt_widget
+#         parent.pack # TODO auto fix the tab folder to avoid having to do this manually (including maintaining bounds and minimum size of shell)
       end
 
       def has_attribute?(attribute_name, *args)
@@ -82,9 +83,11 @@ module Glimmer
       end
       
       def dispose
-        swt_tab_item.setControl(nil)
-        swt_widget.dispose
-        swt_tab_item.dispose
+        auto_exec do
+          swt_tab_item.setControl(nil)
+          swt_widget.dispose
+          swt_tab_item.dispose
+        end
       end
     end
   end

data/lib/glimmer/swt/table_proxy.rb

@@ -266,15 +266,21 @@ module Glimmer
       end
 
       def items
-        swt_widget.get_items
+        auto_exec do
+          swt_widget.get_items
+        end
       end
 
       def model_binding
-        swt_widget.data
+        auto_exec do
+          swt_widget.data
+        end
       end
       
       def table_items_binding
-        swt_widget.get_data('table_items_binding')
+        auto_exec do
+          swt_widget.get_data('table_items_binding')
+        end
       end
       
       def sort_block=(comparator)
@@ -313,7 +319,10 @@ module Glimmer
       
       # Sorts by specified TableColumnProxy object. If nil, it uses the table default sort instead.
       def sort_by_column!(table_column_proxy=nil)
-        index = swt_widget.columns.to_a.index(table_column_proxy.swt_widget) unless table_column_proxy.nil?
+        index = nil
+        auto_exec do
+          index = swt_widget.columns.to_a.index(table_column_proxy.swt_widget) unless table_column_proxy.nil?
+        end
         new_sort_property = table_column_proxy.nil? ? @sort_property : table_column_proxy.sort_property || [column_properties[index]]
         return if table_column_proxy.nil? && new_sort_property.nil? && @sort_block.nil? && @sort_by_block.nil?
         if new_sort_property && table_column_proxy.nil? && new_sort_property.size == 1 && (index = column_sort_properties.index(new_sort_property))
@@ -330,12 +339,16 @@ module Glimmer
         end
         
         @sort_direction = @sort_direction.nil? || @sort_property.first != new_sort_property.first || @sort_direction == :descending ? :ascending : :descending
-        swt_widget.sort_direction = @sort_direction == :ascending ? SWTProxy[:up] : SWTProxy[:down]
+        auto_exec do
+          swt_widget.sort_direction = @sort_direction == :ascending ? SWTProxy[:up] : SWTProxy[:down]
+        end
         
         @sort_property = [new_sort_property].flatten.compact
         table_column_index = column_properties.index(new_sort_property.to_s.to_sym)
         table_column_proxy ||= table_column_proxies[table_column_index] if table_column_index
-        swt_widget.sort_column = table_column_proxy.swt_widget if table_column_proxy
+        auto_exec do
+          swt_widget.sort_column = table_column_proxy.swt_widget if table_column_proxy
+        end
                 
         if table_column_proxy
           @sort_by_block = nil
@@ -405,14 +418,18 @@ module Glimmer
       
       def cells
         column_count = @table.column_properties.size
-        swt_widget.items.map {|item| column_count.times.map {|i| item.get_text(i)} }
+        auto_exec do
+          swt_widget.items.map {|item| column_count.times.map {|i| item.get_text(i)} }
+        end
       end
             
       # Performs a search for table items matching block condition
       # If no condition block is passed, returns all table items
       # Returns a Java TableItem array to easily set as selection on org.eclipse.swt.Table if needed
       def search(&condition)
-        swt_widget.getItems.select {|item| condition.nil? || condition.call(item)}.to_java(TableItem)
+        auto_exec do
+          swt_widget.getItems.select {|item| condition.nil? || condition.call(item)}.to_java(TableItem)
+        end
       end
       
       # Returns all table items including descendants
@@ -454,7 +471,9 @@ module Glimmer
       end
       
       def edit_selected_table_item(column_index, before_write: nil, after_write: nil, after_cancel: nil)
-        edit_table_item(swt_widget.getSelection.first, column_index, before_write: before_write, after_write: after_write, after_cancel: after_cancel)
+        auto_exec do
+          edit_table_item(swt_widget.getSelection.first, column_index, before_write: before_write, after_write: after_write, after_cancel: after_cancel)
+        end
       end
             
       def edit_table_item(table_item, column_index, before_write: nil, after_write: nil, after_cancel: nil, write_on_cancel: false)
@@ -512,7 +531,9 @@ module Glimmer
               model.send("#{model_editing_property}=", new_value) # makes table update itself, so must search for selected table item again
               # Table refresh happens here because of model update triggering observers, so must retrieve table item again
               edited_table_item = search { |ti| ti.getData == model }.first
-              swt_widget.showItem(edited_table_item)
+              auto_exec do
+                swt_widget.showItem(edited_table_item)
+              end
               @table_editor_widget_proxy&.swt_widget&.dispose
               @table_editor_widget_proxy = nil
               if after_write&.arity == 0
@@ -524,7 +545,7 @@ module Glimmer
             end
           end
         end
-
+        
         content {
           @table_editor_widget_proxy = TableProxy::editors.symbolize_keys[editor_widget][:editor_gui].call(editor_widget_args, model, model_editing_property, self)
         }

data/lib/glimmer/swt/transform_proxy.rb

@@ -37,32 +37,36 @@ module Glimmer
       attr_reader :swt_transform, :parent
       
       def initialize(parent, *args, swt_transform: nil, multiply: false)
-        @parent = parent
-        @multiply = multiply
-        if swt_transform.nil?
-          if !args.first.is_a?(Display) && !args.first.is_a?(DisplayProxy)
-            args.prepend DisplayProxy.instance.swt_display
-          end
-          if args.first.is_a?(DisplayProxy)
-            args[0] = args[0].swt_display
-          end
-          if args.last.is_a?(TransformProxy)
-            args[-1] = args[-1].swt_transform
-          end
-          if args.last.nil? || args.last.is_a?(Transform)
-            @swt_transform = args.last
-            @parent&.set_attribute('transform', self)
+        Glimmer::SWT::DisplayProxy.instance.auto_exec do
+          @parent = parent
+          @multiply = multiply
+          if swt_transform.nil?
+            if !args.first.is_a?(Display) && !args.first.is_a?(DisplayProxy)
+              args.prepend DisplayProxy.instance.swt_display
+            end
+            if args.first.is_a?(DisplayProxy)
+              args[0] = args[0].swt_display
+            end
+            if args.last.is_a?(TransformProxy)
+              args[-1] = args[-1].swt_transform
+            end
+            if args.last.nil? || args.last.is_a?(Transform)
+              @swt_transform = args.last
+              @parent&.set_attribute('transform', self)
+            else
+              @swt_transform = Transform.new(*args)
+            end
           else
-            @swt_transform = Transform.new(*args)
+            @swt_transform = swt_transform
           end
-        else
-          @swt_transform = swt_transform
         end
       end
       
       def post_add_content
         if @multiply
-          @parent.multiply(@swt_transform)
+          Glimmer::SWT::DisplayProxy.instance.auto_exec {
+            @parent.multiply(@swt_transform)
+          }
         else
           @parent&.set_attribute('transform', self)
         end
@@ -72,28 +76,12 @@ module Glimmer
         Glimmer::DSL::Engine.add_content(self, Glimmer::DSL::SWT::TransformExpression.new, &block)
       end
       
-      def has_attribute?(attribute_name, *args)
-        @swt_transform.respond_to?(attribute_name) || @swt_transform.respond_to?(attribute_setter(attribute_name))
-      end
-
-      def set_attribute(attribute_name, *args)
-        if @swt_transform.respond_to?(attribute_name)
-          @swt_transform.send(attribute_name, *args)
-        elsif @swt_transform.respond_to?(attribute_setter(attribute_name))
-          @swt_transform.send(attribute_setter(attribute_name), *args)
-        end
-      end
-
-      def get_attribute(attribute_name)
-        if @swt_transform.respond_to?(attribute_getter(attribute_name))
-          @swt_transform.send(attribute_getter(attribute_name))
-        else
-          @swt_transform.send(attribute_name)
-        end
+      def proxy_source_object
+        @swt_transform
       end
       
       def method_missing(method_name, *args, &block)
-        result = @swt_transform.send(method_name, *args, &block)
+        result = Glimmer::SWT::DisplayProxy.instance.auto_exec { @swt_transform.send(method_name, *args, &block) }
         result.nil? ? self : result
       rescue => e
         Glimmer::Config.logger.debug {"Neither MessageBoxProxy nor #{@swt_transform.class.name} can handle the method ##{method}"}

data/lib/glimmer/swt/tree_proxy.rb

@@ -42,7 +42,8 @@ module Glimmer
       # Returns a Java TreeItem array to easily set as selection on org.eclipse.swt.Tree if needed
       def depth_first_search(&condition)
         found = []
-        recursive_depth_first_search(swt_widget.getItems.first, found, &condition)
+        first_item = DisplayProxy.instance.auto_exec { swt_widget.getItems.first }
+        recursive_depth_first_search(first_item, found, &condition)
         found.to_java(TreeItem)
       end
       
@@ -51,18 +52,6 @@ module Glimmer
         depth_first_search
       end
 
-      def widget_property_listener_installers
-        super.merge({
-          Java::OrgEclipseSwtWidgets::Tree => {
-            selection: lambda do |observer|
-              on_widget_selected { |selection_event|
-                observer.call(@swt_widget.getSelection)
-              }
-            end
-          },
-        })
-      end
-      
       def edit_in_progress?
         !!@edit_in_progress
       end
@@ -127,8 +116,10 @@ module Glimmer
 
       def recursive_depth_first_search(tree_item, found, &condition)
         return if tree_item.nil?
-        found << tree_item if condition.nil? || condition.call(tree_item)
-        tree_item.getItems.each do |child_tree_item|
+        found << tree_item if condition.nil? || DisplayProxy.instance.auto_exec {condition.call(tree_item)}
+#         return if found.any? && !has_style?(:multi) # TODO inspect if this is a good optimization when needed
+        tree_items = DisplayProxy.instance.auto_exec {tree_item.getItems}
+        tree_items.each do |child_tree_item|
           recursive_depth_first_search(child_tree_item, found, &condition)
         end
       end
@@ -136,7 +127,11 @@ module Glimmer
       def property_type_converters
         super.merge({
           selection: lambda do |value|
-            depth_first_search {|ti| ti.getData == value}
+            if value.is_a?(Array)
+              depth_first_search {|ti| value.include?(ti.getData) }
+            else
+              depth_first_search {|ti| ti.getData == value}
+            end
           end,
         })
       end

data/lib/glimmer/swt/widget_listener_proxy.rb

@@ -53,9 +53,13 @@ module Glimmer
             @swt_display.removeListener(@event_type, @swt_listener)
           end
         elsif @event_type
-          @swt_widget.removeListener(@event_type, @swt_listener)
+          DisplayProxy.instance.auto_exec do
+            @swt_widget.removeListener(@event_type, @swt_listener)
+          end
         else
-          @swt_widget.send(widget_remove_listener_method, @swt_listener)
+          DisplayProxy.instance.auto_exec do
+            @swt_widget.send(widget_remove_listener_method, @swt_listener)
+          end
         end
       end
       alias unregister deregister # TODO consider dropping unregister (and in Observer too)

data/lib/glimmer/swt/widget_proxy.rb

@@ -26,7 +26,7 @@ require 'glimmer/swt/swt_proxy'
 require 'glimmer/swt/display_proxy'
 require 'glimmer/swt/dnd_proxy'
 require 'glimmer/swt/image_proxy'
-require 'glimmer/swt/properties'
+require 'glimmer/swt/proxy_properties'
 require 'glimmer/swt/custom/drawable'
 
 # TODO refactor to make file smaller and extract sub-widget-proxies out of this
@@ -44,13 +44,13 @@ module Glimmer
     # Follows the Proxy Design Pattern
     class WidgetProxy
       include Packages
-      include Properties
+      include ProxyProperties
       include Custom::Drawable
 
       DEFAULT_STYLES = {
         'arrow'               => [:arrow],
         'button'              => [:push],
-        'canvas'              => [:double_buffered],
+        'canvas'              => ([:double_buffered] if OS.windows?),
         'checkbox'            => [:check],
         'check'               => [:check],
         'drag_source'         => [:drop_copy],
@@ -112,13 +112,15 @@ module Glimmer
         # Instantiates the right WidgetProxy subclass for passed in keyword
         # Args are: keyword, parent, swt_widget_args (including styles)
         def create(*init_args, swt_widget: nil)
-          return swt_widget.get_data('proxy') if swt_widget&.get_data('proxy')
-          keyword, parent, args = init_args
-          selected_widget_proxy_class = widget_proxy_class(keyword || underscored_widget_name(swt_widget))
-          if init_args.empty?
-            selected_widget_proxy_class.new(swt_widget: swt_widget)
-          else
-            selected_widget_proxy_class.new(*init_args)
+          DisplayProxy.instance.auto_exec do
+            return swt_widget.get_data('proxy') if swt_widget&.get_data('proxy')
+            keyword, parent, args = init_args
+            selected_widget_proxy_class = widget_proxy_class(keyword || underscored_widget_name(swt_widget))
+            if init_args.empty?
+              selected_widget_proxy_class.new(swt_widget: swt_widget)
+            else
+              selected_widget_proxy_class.new(*init_args)
+            end
           end
         end
         
@@ -149,31 +151,33 @@ module Glimmer
       #
       # Styles is a comma separate list of symbols representing SWT styles in lower case
       def initialize(*init_args, swt_widget: nil)
-        @image_double_buffered = !!(init_args&.last&.include?(:image_double_buffered) && init_args&.last&.delete(:image_double_buffered))
-        if swt_widget.nil?
-          underscored_widget_name, parent, args = init_args
-          @parent_proxy = parent
-          styles, extra_options = extract_args(underscored_widget_name, args)
-          swt_widget_class = self.class.swt_widget_class_for(underscored_widget_name)
-          @swt_widget = swt_widget_class.new(@parent_proxy.swt_widget, style(underscored_widget_name, styles), *extra_options)
-        else
-          @swt_widget = swt_widget
-          underscored_widget_name = self.class.underscored_widget_name(@swt_widget)
-          parent_proxy_class = self.class.widget_proxy_class(self.class.underscored_widget_name(@swt_widget.parent))
-          parent = swt_widget.parent
-          @parent_proxy = parent&.get_data('proxy') || parent_proxy_class.new(swt_widget: parent)
-        end
-        if @swt_widget&.get_data('proxy').nil?
-          @swt_widget.set_data('proxy', self)
-          DEFAULT_INITIALIZERS[underscored_widget_name.to_s.to_sym]&.call(@swt_widget)
-          @parent_proxy.post_initialize_child(self)
-        end
-        @keyword = underscored_widget_name.to_s
-        if respond_to?(:on_widget_disposed)
-          on_widget_disposed {
-            clear_shapes
-            deregister_shape_painting
-          }
+        auto_exec do
+          @image_double_buffered = !!(init_args&.last&.include?(:image_double_buffered) && init_args&.last&.delete(:image_double_buffered))
+          if swt_widget.nil?
+            underscored_widget_name, parent, args = init_args
+            @parent_proxy = parent
+            styles, extra_options = extract_args(underscored_widget_name, args)
+            swt_widget_class = self.class.swt_widget_class_for(underscored_widget_name)
+            @swt_widget = swt_widget_class.new(@parent_proxy.swt_widget, style(underscored_widget_name, styles), *extra_options)
+          else
+            @swt_widget = swt_widget
+            underscored_widget_name = self.class.underscored_widget_name(@swt_widget)
+            parent_proxy_class = self.class.widget_proxy_class(self.class.underscored_widget_name(@swt_widget.parent))
+            parent = swt_widget.parent
+            @parent_proxy = parent&.get_data('proxy') || parent_proxy_class.new(swt_widget: parent)
+          end
+          if @swt_widget&.get_data('proxy').nil?
+            @swt_widget.set_data('proxy', self)
+            DEFAULT_INITIALIZERS[underscored_widget_name.to_s.to_sym]&.call(@swt_widget)
+            @parent_proxy.post_initialize_child(self)
+          end
+          @keyword = underscored_widget_name.to_s
+          if respond_to?(:on_widget_disposed)
+            on_widget_disposed {
+              clear_shapes
+              deregister_shape_painting
+            }
+          end
         end
       end
       
@@ -219,81 +223,77 @@ module Glimmer
           [args, extra_options]
         end
       end
-
-      def has_attribute_getter?(attribute_getter_name, *args)
-        attribute_getter_name = attribute_getter_name.to_s.underscore
-        return false unless !attribute_getter_name.end_with?('=') && !attribute_getter_name.start_with?('set_')
-        args.empty? && swt_widget.respond_to?(attribute_getter_name)
-      end
       
-      def has_attribute_setter?(attribute_setter_name, *args)
-        attribute_setter_name = attribute_setter_name.to_s
-        underscored_attribute_setter_name = attribute_setter_name.underscore
-        return false unless attribute_setter_name.end_with?('=') || (attribute_setter_name.start_with?('set_') && !args.empty?)
-        attribute_name = underscored_attribute_setter_name.sub(/^set_/, '').sub(/=$/, '')
-        has_attribute?(attribute_name, *args)
+      def proxy_source_object
+        @swt_widget
       end
-      
+
       def has_attribute?(attribute_name, *args)
         # TODO test that attribute getter responds too
         widget_custom_attribute = widget_custom_attribute_mapping[attribute_name.to_s]
-        if widget_custom_attribute
-          @swt_widget.respond_to?(widget_custom_attribute[:setter][:name])
-        else
-          @swt_widget.respond_to?(attribute_setter(attribute_name), args) || respond_to?(ruby_attribute_setter(attribute_name), args)
+        auto_exec do
+          if widget_custom_attribute
+            @swt_widget.respond_to?(widget_custom_attribute[:setter][:name])
+          else
+            super
+          end
         end
       end
 
       def set_attribute(attribute_name, *args)
         # TODO Think about widget subclasses overriding set_attribute to add more attributes vs adding as Ruby attributes directly
         widget_custom_attribute = widget_custom_attribute_mapping[attribute_name.to_s]
-        apply_property_type_converters(normalized_attribute(attribute_name), args)
-        if widget_custom_attribute
-          widget_custom_attribute[:setter][:invoker].call(@swt_widget, args)
-        elsif @swt_widget.respond_to?(attribute_setter(attribute_name))
-          @swt_widget.send(attribute_setter(attribute_name), *args) unless @swt_widget.send(attribute_getter(attribute_name)) == args.first
-        elsif @swt_widget.respond_to?(ruby_attribute_setter(attribute_name))
-          @swt_widget.send(ruby_attribute_setter(attribute_name), args)
-        else
-          send(ruby_attribute_setter(attribute_name), args)
+        auto_exec do
+          apply_property_type_converters(normalized_attribute(attribute_name), args)
         end
+        swt_widget_operation = false
+        result = nil
+        auto_exec do
+          result = if widget_custom_attribute
+            swt_widget_operation = true
+            widget_custom_attribute[:setter][:invoker].call(@swt_widget, args)
+          end
+        end
+        result = super unless swt_widget_operation
+        result
       end
 
       def get_attribute(attribute_name)
         widget_custom_attribute = widget_custom_attribute_mapping[attribute_name.to_s]
-        if widget_custom_attribute
-          if widget_custom_attribute[:getter][:invoker]
-            widget_custom_attribute[:getter][:invoker].call(@swt_widget, [])
-          else
-            @swt_widget.send(widget_custom_attribute[:getter][:name])
+        swt_widget_operation = false
+        result = nil
+        auto_exec do
+          result = if widget_custom_attribute
+            swt_widget_operation = true
+            if widget_custom_attribute[:getter][:invoker]
+              widget_custom_attribute[:getter][:invoker].call(@swt_widget, [])
+            else
+              @swt_widget.send(widget_custom_attribute[:getter][:name])
+            end
           end
-        elsif @swt_widget.respond_to?(attribute_getter(attribute_name))
-          @swt_widget.send(attribute_getter(attribute_name))
-        elsif @swt_widget.respond_to?(ruby_attribute_getter(attribute_name))
-          @swt_widget.send(ruby_attribute_getter(attribute_name))
-        elsif @swt_widget.respond_to?(attribute_name)
-          @swt_widget.send(attribute_name)
-        elsif respond_to?(ruby_attribute_getter(attribute_name))
-          send(ruby_attribute_getter(attribute_name))
-        else
-          send(attribute_name)
         end
+        result = super unless swt_widget_operation
+        result
       end
 
       def pack_same_size
-        bounds = @swt_widget.getBounds
-        listener = on_control_resized {
-          @swt_widget.setSize(bounds.width, bounds.height)
-          @swt_widget.setLocation(bounds.x, bounds.y)
-        }
-        if @swt_widget.is_a?(Composite)
-          @swt_widget.layout(true, true)
-        else
-          @swt_widget.pack(true)
+        auto_exec do
+          bounds = @swt_widget.getBounds
+          listener = on_control_resized {
+            @swt_widget.setSize(bounds.width, bounds.height)
+            @swt_widget.setLocation(bounds.x, bounds.y)
+          }
+          if @swt_widget.is_a?(Composite)
+            @swt_widget.layout(true, true)
+          else
+            @swt_widget.pack(true)
+          end
+          @swt_widget.removeControlListener(listener.swt_listener)
         end
-        @swt_widget.removeControlListener(listener.swt_listener)
       end
 
+      # these work in tandem with the property_type_converters
+      # sometimes, they are specified in subclasses instead
       def widget_property_listener_installers
         @swt_widget_property_listener_installers ||= {
           Java::OrgEclipseSwtWidgets::Control => {
@@ -334,6 +334,17 @@ module Glimmer
               }
             end,
           },
+          Java::OrgEclipseSwtWidgets::Tree => {
+            :selection => lambda do |observer|
+              on_widget_selected { |selection_event|
+                if has_style?(:multi)
+                  observer.call(@swt_widget.getSelection.map(&:get_data))
+                else
+                  observer.call(@swt_widget.getSelection.first&.get_data)
+                end
+              }
+            end,
+          },
           Java::OrgEclipseSwtWidgets::Text => {
             :text => lambda do |observer|
               on_modify_text { |modify_event|
@@ -546,25 +557,37 @@ module Glimmer
         @flyweight_swt_widget_classes ||= {}
       end
 
+      # delegates to DisplayProxy
       def async_exec(&block)
         DisplayProxy.instance.async_exec(&block)
       end
 
+      # delegates to DisplayProxy
       def sync_exec(&block)
         DisplayProxy.instance.sync_exec(&block)
       end
 
+      # delegates to DisplayProxy
       def timer_exec(delay_in_millis, &block)
         DisplayProxy.instance.timer_exec(delay_in_millis, &block)
       end
 
+      # delegates to DisplayProxy
+      def auto_exec(override_sync_exec: nil, override_async_exec: nil, &block)
+        DisplayProxy.instance.auto_exec(override_sync_exec: override_sync_exec, override_async_exec: override_async_exec, &block)
+      end
+
       def has_style?(style)
         comparison = interpret_style(style)
-        (@swt_widget.style & comparison) == comparison
+        auto_exec do
+          (@swt_widget.style & comparison) == comparison
+        end
       end
 
       def dispose
-        @swt_widget.dispose
+        auto_exec do
+          @swt_widget.dispose
+        end
       end
 
       def disposed?
@@ -574,15 +597,19 @@ module Glimmer
       # TODO Consider renaming these methods as they are mainly used for data-binding
 
       def can_add_observer?(property_name)
-        @swt_widget.class.ancestors.map {|ancestor| widget_property_listener_installers[ancestor]}.compact.map(&:keys).flatten.map(&:to_s).include?(property_name.to_s)
+        auto_exec do
+          @swt_widget.class.ancestors.map {|ancestor| widget_property_listener_installers[ancestor]}.compact.map(&:keys).flatten.map(&:to_s).include?(property_name.to_s)
+        end
       end
 
       # Used for data-binding only. Consider renaming or improving to avoid the confusion it causes
       def add_observer(observer, property_name)
         if !observer.respond_to?(:binding_options) || !observer.binding_options[:read_only]
-          property_listener_installers = @swt_widget.class.ancestors.map {|ancestor| widget_property_listener_installers[ancestor]}.compact
-          widget_listener_installers = property_listener_installers.map{|installer| installer[property_name.to_s.to_sym]}.compact if !property_listener_installers.empty?
-          widget_listener_installers.to_a.first&.call(observer)
+          auto_exec do
+            property_listener_installers = @swt_widget.class.ancestors.map {|ancestor| widget_property_listener_installers[ancestor]}.compact
+            widget_listener_installers = property_listener_installers.map{|installer| installer[property_name.to_s.to_sym]}.compact if !property_listener_installers.empty?
+            widget_listener_installers.to_a.first&.call(observer)
+          end
         end
       end
 
@@ -597,7 +624,7 @@ module Glimmer
       end
       
       def ensure_drop_target_proxy(style=[])
-        @drop_target_proxy ||= self.class.new('drop_target', self, style).tap do |proxy|
+        @drop_target_proxy ||= WidgetProxy.new('drop_target', self, style).tap do |proxy|
           proxy.set_attribute(:transfer, :text)
           proxy.on_drag_enter { |event|
             event.detail = DNDProxy[:drop_copy]
@@ -608,7 +635,7 @@ module Glimmer
       # TODO eliminate duplication in the following methods perhaps by relying on exceptions
 
       def can_handle_observation_request?(observation_request)
-        observation_request = observation_request.to_s
+        observation_request = normalize_observation_request(observation_request)
         if observation_request.start_with?('on_swt_')
           constant_name = observation_request.sub(/^on_swt_/, '')
           SWTProxy.has_constant?(constant_name)
@@ -619,13 +646,15 @@ module Glimmer
       end
       
       def can_handle_drag_observation_request?(observation_request)
-        return false unless swt_widget.is_a?(Control)
-        potential_drag_source = @drag_source_proxy.nil?
-        ensure_drag_source_proxy
-        @drag_source_proxy.can_handle_observation_request?(observation_request).tap do |result|
-          if potential_drag_source && !result
-            @drag_source_proxy.swt_widget.dispose
-            @drag_source_proxy = nil
+        auto_exec do
+          return false unless swt_widget.is_a?(Control)
+          potential_drag_source = @drag_source_proxy.nil?
+          ensure_drag_source_proxy
+          @drag_source_proxy.can_handle_observation_request?(observation_request).tap do |result|
+            if potential_drag_source && !result
+              @drag_source_proxy.swt_widget.dispose
+              @drag_source_proxy = nil
+            end
           end
         end
       rescue => e
@@ -634,19 +663,21 @@ module Glimmer
       end
 
       def can_handle_drop_observation_request?(observation_request)
-        return false unless swt_widget.is_a?(Control)
-        potential_drop_target = @drop_target_proxy.nil?
-        ensure_drop_target_proxy
-        @drop_target_proxy.can_handle_observation_request?(observation_request).tap do |result|
-          if potential_drop_target && !result
-            @drop_target_proxy.swt_widget.dispose
-            @drop_target_proxy = nil
+        auto_exec do
+          return false unless swt_widget.is_a?(Control)
+          potential_drop_target = @drop_target_proxy.nil?
+          ensure_drop_target_proxy
+          @drop_target_proxy.can_handle_observation_request?(observation_request).tap do |result|
+            if potential_drop_target && !result
+              @drop_target_proxy.swt_widget.dispose
+              @drop_target_proxy = nil
+            end
           end
         end
       end
 
       def handle_observation_request(observation_request, &block)
-        observation_request = observation_request.to_s
+        observation_request = normalize_observation_request(observation_request)
         if observation_request.start_with?('on_swt_')
           constant_name = observation_request.sub(/^on_swt_/, '')
           add_swt_event_listener(constant_name, &block)
@@ -664,30 +695,26 @@ module Glimmer
       end
 
       def content(&block)
-        Glimmer::DSL::Engine.add_content(self, Glimmer::DSL::SWT::WidgetExpression.new, &block)
+        auto_exec do
+          Glimmer::DSL::Engine.add_content(self, Glimmer::DSL::SWT::WidgetExpression.new, &block)
+        end
       end
 
       def method_missing(method, *args, &block)
+        # TODO push most of this logic down to Properties (and perhaps create Listeners module as well)
         if can_handle_observation_request?(method)
           handle_observation_request(method, &block)
-        elsif has_attribute_setter?(method, *args)
-          set_attribute(method, *args)
-        elsif has_attribute_getter?(method, *args)
-          get_attribute(method, *args)
         else
-          swt_widget.send(method, *args, &block)
+          super
         end
-      rescue => e
-        Glimmer::Config.logger.debug { "Neither WidgetProxy nor #{swt_widget.class.name} can handle the method ##{method}" }
-        Glimmer::Config.logger.debug { e.full_message }
-        super
-        # TODO consider get_attribute too
       end
       
       def respond_to?(method, *args, &block)
-        super ||
-          can_handle_observation_request?(method) ||
-          swt_widget.respond_to?(method, *args, &block)
+        result = super
+        return true if result
+        auto_exec do
+          can_handle_observation_request?(method)
+        end
       end
       
       private
@@ -713,16 +740,26 @@ module Glimmer
       # add_listener knowing it will be called for sure afterwards
 
       def can_add_listener?(underscored_listener_name)
-        !self.class.find_listener(@swt_widget.getClass, underscored_listener_name).empty?
+        auto_exec do
+          !self.class.find_listener(@swt_widget.getClass, underscored_listener_name).empty?
+        end
       end
 
       def add_listener(underscored_listener_name, &block)
-        widget_add_listener_method, listener_class, listener_method = self.class.find_listener(@swt_widget.getClass, underscored_listener_name)
-        widget_listener_proxy = nil
-        safe_block = lambda { |*args| block.call(*args) unless @swt_widget.isDisposed }
-        listener = listener_class.new(listener_method => safe_block)
-        @swt_widget.send(widget_add_listener_method, listener)
-        WidgetListenerProxy.new(swt_widget: @swt_widget, swt_listener: listener, widget_add_listener_method: widget_add_listener_method, swt_listener_class: listener_class, swt_listener_method: listener_method)
+        auto_exec do
+          widget_add_listener_method, listener_class, listener_method = self.class.find_listener(@swt_widget.getClass, underscored_listener_name)
+          widget_listener_proxy = nil
+          safe_block = lambda do |*args|
+            begin
+              block.call(*args) unless @swt_widget.isDisposed
+            rescue => e
+              Glimmer::Config.logger.error {e}
+            end
+          end
+          listener = listener_class.new(listener_method => safe_block)
+          @swt_widget.send(widget_add_listener_method, listener)
+          WidgetListenerProxy.new(swt_widget: @swt_widget, swt_listener: listener, widget_add_listener_method: widget_add_listener_method, swt_listener_class: listener_class, swt_listener_method: listener_method)
+        end
       end
 
       # Looks through SWT class add***Listener methods till it finds one for which
@@ -772,16 +809,22 @@ module Glimmer
       end
 
       def add_swt_event_listener(swt_constant, &block)
-        event_type = SWTProxy[swt_constant]
-        widget_listener_proxy = nil
-        safe_block = lambda { |*args| block.call(*args) unless @swt_widget.isDisposed }
-        @swt_widget.addListener(event_type, &safe_block)
-        widget_listener_proxy = WidgetListenerProxy.new(swt_widget: @swt_widget, swt_listener: @swt_widget.getListeners(event_type).last, event_type: event_type, swt_constant: swt_constant)
+        auto_exec do
+          event_type = SWTProxy[swt_constant]
+          widget_listener_proxy = nil
+          safe_block = lambda { |*args| block.call(*args) unless @swt_widget.isDisposed }
+          @swt_widget.addListener(event_type, &safe_block)
+          widget_listener_proxy = WidgetListenerProxy.new(swt_widget: @swt_widget, swt_listener: @swt_widget.getListeners(event_type).last, event_type: event_type, swt_constant: swt_constant)
+        end
       end
 
       def widget_custom_attribute_mapping
         # TODO scope per widget class type just like other mappings
         @swt_widget_custom_attribute_mapping ||= {
+          'window' => {
+            getter: {name: 'getShell'},
+            setter: {name: 'getShell', invoker: lambda { |widget, args| @swt_widget.getShell }}, # No Op
+          },
           'focus' => {
             getter: {name: 'isFocusControl'},
             setter: {name: 'setFocus', invoker: lambda { |widget, args| @swt_widget.setFocus if args.first }},
@@ -834,6 +877,14 @@ module Glimmer
         ensure_drop_target_proxy
         @drop_target_proxy.set_attribute(:drop_target_effect, args)
       end
+      
+      def normalize_observation_request(observation_request)
+        observation_request = observation_request.to_s
+        if @swt_widget.is_a?(Shell) && observation_request.downcase.include?('window')
+          observation_request = observation_request.sub('window', 'shell').sub('Window', 'Shell')
+        end
+        observation_request
+      end
 
       def apply_property_type_converters(attribute_name, args)
         value = args
@@ -849,6 +900,8 @@ module Glimmer
         color_converter = lambda do |value|
           if value.is_a?(Symbol) || value.is_a?(String)
             ColorProxy.new(value).swt_color
+          elsif value.is_a?(RGB)
+            ColorProxy.new(value.red, value.green, value.blue).swt_color
           else
             value
           end
@@ -887,7 +940,7 @@ module Glimmer
                 while last_image_number != image_number
                   last_image_number = image_number
                   sync_exec {
-                    redraw
+                    redraw unless disposed?
                   }
                   delayTime = loader.data[image_number].delayTime.to_f / 100.0
                   sleep(delayTime)
@@ -918,7 +971,7 @@ module Glimmer
           foreground: color_converter,
           link_foreground: color_converter,
           font: lambda do |value|
-            if value.is_a?(Hash)
+            if value.is_a?(Hash) || value.is_a?(FontData)
               font_properties = value
               FontProxy.new(self, font_properties).swt_font
             else
@@ -936,6 +989,10 @@ module Glimmer
           items: lambda do |value|
             value.to_java :string
           end,
+          selection: lambda do |value|
+            value = value.to_f if swt_widget.is_a?(Spinner)
+            value
+          end,
           text: lambda do |value|
             value.to_s
           end,