glimmer-dsl-swt 4.18.4.8 → 4.18.5.1

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.8 ruby lib
+# stub: glimmer-dsl-swt 4.18.5.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "glimmer-dsl-swt".freeze
-  s.version = "4.18.4.8"
+  s.version = "4.18.5.1"
 
   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-23"
   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",
@@ -52,6 +58,7 @@ Gem::Specification.new do |s|
     "lib/glimmer/data_binding/widget_binding.rb",
     "lib/glimmer/dsl/swt/animation_expression.rb",
     "lib/glimmer/dsl/swt/async_exec_expression.rb",
+    "lib/glimmer/dsl/swt/auto_exec_expression.rb",
     "lib/glimmer/dsl/swt/bind_expression.rb",
     "lib/glimmer/dsl/swt/block_property_expression.rb",
     "lib/glimmer/dsl/swt/checkbox_group_selection_data_binding_expression.rb",
@@ -62,13 +69,11 @@ Gem::Specification.new do |s|
     "lib/glimmer/dsl/swt/custom_widget_expression.rb",
     "lib/glimmer/dsl/swt/data_binding_expression.rb",
     "lib/glimmer/dsl/swt/dialog_expression.rb",
-    "lib/glimmer/dsl/swt/directory_dialog_expression.rb",
     "lib/glimmer/dsl/swt/display_expression.rb",
     "lib/glimmer/dsl/swt/dnd_expression.rb",
     "lib/glimmer/dsl/swt/dsl.rb",
     "lib/glimmer/dsl/swt/exec_expression.rb",
     "lib/glimmer/dsl/swt/expand_item_expression.rb",
-    "lib/glimmer/dsl/swt/file_dialog_expression.rb",
     "lib/glimmer/dsl/swt/font_expression.rb",
     "lib/glimmer/dsl/swt/image_expression.rb",
     "lib/glimmer/dsl/swt/layout_data_expression.rb",
@@ -109,12 +114,21 @@ Gem::Specification.new do |s|
     "lib/glimmer/swt/custom/drawable.rb",
     "lib/glimmer/swt/custom/radio_group.rb",
     "lib/glimmer/swt/custom/shape.rb",
+    "lib/glimmer/swt/custom/shape/arc.rb",
+    "lib/glimmer/swt/custom/shape/focus.rb",
+    "lib/glimmer/swt/custom/shape/image.rb",
+    "lib/glimmer/swt/custom/shape/line.rb",
+    "lib/glimmer/swt/custom/shape/oval.rb",
+    "lib/glimmer/swt/custom/shape/point.rb",
+    "lib/glimmer/swt/custom/shape/polygon.rb",
+    "lib/glimmer/swt/custom/shape/polyline.rb",
+    "lib/glimmer/swt/custom/shape/rectangle.rb",
+    "lib/glimmer/swt/custom/shape/text.rb",
     "lib/glimmer/swt/date_time_proxy.rb",
-    "lib/glimmer/swt/directory_dialog_proxy.rb",
+    "lib/glimmer/swt/dialog_proxy.rb",
     "lib/glimmer/swt/display_proxy.rb",
     "lib/glimmer/swt/dnd_proxy.rb",
     "lib/glimmer/swt/expand_item_proxy.rb",
-    "lib/glimmer/swt/file_dialog_proxy.rb",
     "lib/glimmer/swt/font_proxy.rb",
     "lib/glimmer/swt/image_proxy.rb",
     "lib/glimmer/swt/layout_data_proxy.rb",
@@ -123,6 +137,7 @@ Gem::Specification.new do |s|
     "lib/glimmer/swt/message_box_proxy.rb",
     "lib/glimmer/swt/packages.rb",
     "lib/glimmer/swt/properties.rb",
+    "lib/glimmer/swt/proxy_properties.rb",
     "lib/glimmer/swt/sash_form_proxy.rb",
     "lib/glimmer/swt/scrolled_composite_proxy.rb",
     "lib/glimmer/swt/shell_proxy.rb",
@@ -136,6 +151,7 @@ Gem::Specification.new do |s|
     "lib/glimmer/swt/tree_proxy.rb",
     "lib/glimmer/swt/widget_listener_proxy.rb",
     "lib/glimmer/swt/widget_proxy.rb",
+    "lib/glimmer/ui.rb",
     "lib/glimmer/ui/custom_shell.rb",
     "lib/glimmer/ui/custom_widget.rb",
     "lib/glimmer/util/proc_tracker.rb",
@@ -168,9 +184,11 @@ Gem::Specification.new do |s|
     "samples/hello/hello_checkbox.rb",
     "samples/hello/hello_checkbox_group.rb",
     "samples/hello/hello_code_text.rb",
+    "samples/hello/hello_color_dialog.rb",
     "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",
@@ -179,6 +197,7 @@ Gem::Specification.new do |s|
     "samples/hello/hello_drag_and_drop.rb",
     "samples/hello/hello_expand_bar.rb",
     "samples/hello/hello_file_dialog.rb",
+    "samples/hello/hello_font_dialog.rb",
     "samples/hello/hello_group.rb",
     "samples/hello/hello_link.rb",
     "samples/hello/hello_list_multi_selection.rb",
@@ -186,6 +205,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",
@@ -194,6 +214,7 @@ Gem::Specification.new do |s|
     "samples/hello/hello_tab.rb",
     "samples/hello/hello_table.rb",
     "samples/hello/hello_table/baseball_park.png",
+    "samples/hello/hello_tree.rb",
     "samples/hello/hello_world.rb",
     "vendor/swt/linux/swt.jar",
     "vendor/swt/mac/swt.jar",
@@ -209,12 +230,11 @@ Gem::Specification.new do |s|
     s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<glimmer>.freeze, ["~> 1.1.1"])
+      s.add_runtime_dependency(%q<glimmer>.freeze, ["~> 1.1.2"])
       s.add_runtime_dependency(%q<super_module>.freeze, [">= 1.4.1", "< 2.0.0"])
       s.add_runtime_dependency(%q<nested_inherited_jruby_include_package>.freeze, [">= 0.3.0", "< 2.0.0"])
       s.add_runtime_dependency(%q<puts_debuggerer>.freeze, [">= 0.11.0", "< 2.0.0"])
       s.add_runtime_dependency(%q<rake-tui>.freeze, [">= 0.2.3", "< 2.0.0"])
-      s.add_runtime_dependency(%q<git>.freeze, ["= 1.8.1"])
       s.add_runtime_dependency(%q<logging>.freeze, [">= 2.3.0", "< 3.0.0"])
       s.add_runtime_dependency(%q<os>.freeze, [">= 1.0.0", "< 2.0.0"])
       s.add_runtime_dependency(%q<rake>.freeze, [">= 10.1.0", "< 14.0.0"])
@@ -228,12 +248,11 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<simplecov>.freeze, ["~> 0.16.1"])
       s.add_development_dependency(%q<simplecov-lcov>.freeze, ["~> 0.7.0"])
     else
-      s.add_dependency(%q<glimmer>.freeze, ["~> 1.1.1"])
+      s.add_dependency(%q<glimmer>.freeze, ["~> 1.1.2"])
       s.add_dependency(%q<super_module>.freeze, [">= 1.4.1", "< 2.0.0"])
       s.add_dependency(%q<nested_inherited_jruby_include_package>.freeze, [">= 0.3.0", "< 2.0.0"])
       s.add_dependency(%q<puts_debuggerer>.freeze, [">= 0.11.0", "< 2.0.0"])
       s.add_dependency(%q<rake-tui>.freeze, [">= 0.2.3", "< 2.0.0"])
-      s.add_dependency(%q<git>.freeze, ["= 1.8.1"])
       s.add_dependency(%q<logging>.freeze, [">= 2.3.0", "< 3.0.0"])
       s.add_dependency(%q<os>.freeze, [">= 1.0.0", "< 2.0.0"])
       s.add_dependency(%q<rake>.freeze, [">= 10.1.0", "< 14.0.0"])
@@ -248,12 +267,11 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<simplecov-lcov>.freeze, ["~> 0.7.0"])
     end
   else
-    s.add_dependency(%q<glimmer>.freeze, ["~> 1.1.1"])
+    s.add_dependency(%q<glimmer>.freeze, ["~> 1.1.2"])
     s.add_dependency(%q<super_module>.freeze, [">= 1.4.1", "< 2.0.0"])
     s.add_dependency(%q<nested_inherited_jruby_include_package>.freeze, [">= 0.3.0", "< 2.0.0"])
     s.add_dependency(%q<puts_debuggerer>.freeze, [">= 0.11.0", "< 2.0.0"])
     s.add_dependency(%q<rake-tui>.freeze, [">= 0.2.3", "< 2.0.0"])
-    s.add_dependency(%q<git>.freeze, ["= 1.8.1"])
     s.add_dependency(%q<logging>.freeze, [">= 2.3.0", "< 3.0.0"])
     s.add_dependency(%q<os>.freeze, [">= 1.0.0", "< 2.0.0"])
     s.add_dependency(%q<rake>.freeze, [">= 10.1.0", "< 14.0.0"])

data/lib/ext/glimmer/config.rb

@@ -32,8 +32,7 @@ module Glimmer
       'org.eclipse.swt.custom',
       'org.eclipse.swt.dnd',
     ]
-    DEFAULT_AUTO_SYNC_EXEC = false
-    GUI_THREAD = Thread.current
+    DEFAULT_AUTO_SYNC_EXEC = true
     
     class << self
       # Tells Glimmer to import SWT packages into including class (default: true)
@@ -59,10 +58,6 @@ module Glimmer
       end
       alias auto_sync_exec? auto_sync_exec
       
-      def require_sync_exec?
-        Thread.current != GUI_THREAD
-      end
-      
       # Returns Logging Devices. Default is [:stdout, :syslog]
       def logging_devices
         unless defined? @@logging_devices
@@ -147,9 +142,10 @@ end
 Glimmer::Config.reset_logger! unless ENV['GLIMMER_LOGGER_ENABLED'].to_s.downcase == 'false'
 if ENV['GLIMMER_LOGGER_LEVEL']
   # if glimmer log level is being overridden for debugging purposes, then disable async logging making logging immediate
-  Glimmer::Config.logging_appender_options = Glimmer::Config.logging_appender_options.merge(async: false, auto_flushing: 1)
+  Glimmer::Config.logging_appender_options = Glimmer::Config.logging_appender_options.merge(async: false, auto_flushing: 1, immediate_at: [:unknown, :debug, :info, :error, :fatal])
   Glimmer::Config.logging_devices = [:stdout]
   begin
+    puts "Adjusting Glimmer logging level to #{ENV['GLIMMER_LOGGER_LEVEL']}"
     Glimmer::Config.logger.level = ENV['GLIMMER_LOGGER_LEVEL'].strip
   rescue => e
     puts e.message

data/lib/glimmer/data_binding/list_selection_binding.rb

@@ -1,5 +1,5 @@
 # Copyright (c) 2007-2021 Andy Maleh
-# 
+#
 # Permission is hereby granted, free of charge, to any person obtaining
 # a copy of this software and associated documentation files (the
 # "Software"), to deal in the Software without restriction, including
@@ -7,10 +7,10 @@
 # distribute, sublicense, and/or sell copies of the Software, and to
 # permit persons to whom the Software is furnished to do so, subject to
 # the following conditions:
-# 
+#
 # The above copyright notice and this permission notice shall be
 # included in all copies or substantial portions of the Software.
-# 
+#
 # THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
 # EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
 # MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
@@ -50,17 +50,23 @@ module Glimmer
       # Initialize with list widget and property_type
       # property_type :string represents default list single selection
       # property_type :array represents list multi selection
-      def initialize(widget_proxy, property_type)
+      def initialize(widget_proxy, property_type, sync_exec: false, async_exec: false)
         property_type = :string if property_type.nil? or property_type == :undefined
         @widget_proxy = widget_proxy
         @property_type = property_type
-        @widget_proxy.on_widget_disposed do |dispose_event|
-          unregister_all_observables
+        @sync_exec = sync_exec
+        @async_exec = async_exec
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @sync_exec, override_async_exec: @async_exec) do
+          @widget_proxy.on_widget_disposed do |dispose_event|
+            unregister_all_observables
+          end
         end
       end
 
       def call(value)
-        PROPERTY_TYPE_UPDATERS[@property_type].call(@widget_proxy, value) unless evaluate_property == value
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @sync_exec, override_async_exec: @async_exec) do
+          PROPERTY_TYPE_UPDATERS[@property_type].call(@widget_proxy, value) unless evaluate_property == value
+        end
       end
 
       def evaluate_property

data/lib/glimmer/data_binding/table_items_binding.rb

@@ -24,6 +24,7 @@ require 'glimmer/data_binding/observable_model'
 require 'glimmer/data_binding/observable'
 require 'glimmer/data_binding/observer'
 require 'glimmer/swt/swt_proxy'
+require 'glimmer/swt/display_proxy'
 
 module Glimmer
   module DataBinding
@@ -37,33 +38,37 @@ module Glimmer
         @table = parent
         @model_binding = model_binding
         @read_only_sort = @model_binding.binding_options[:read_only_sort]
-        @table.swt_widget.data = @model_binding
-        @table.swt_widget.set_data('table_items_binding', self)
         @column_properties = column_properties
-        @table.on_widget_disposed do |dispose_event|
-          unregister_all_observables
-        end
         if @table.respond_to?(:column_properties=)
           @table.column_properties = @column_properties
         else # assume custom widget
           @table.body_root.column_properties = @column_properties
         end
-        @table_observer_registration = observe(model_binding)
-        call
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @model_binding.binding_options[:sync_exec], override_async_exec: @model_binding.binding_options[:async_exec]) do
+          @table.swt_widget.data = @model_binding
+          @table.swt_widget.set_data('table_items_binding', self)
+          @table.on_widget_disposed do |dispose_event|
+            unregister_all_observables
+          end
+          @table_observer_registration = observe(model_binding)
+          call
+        end
       end
 
       def call(new_model_collection=nil, internal_sort: false)
-        new_model_collection = model_binding_evaluated_property = @model_binding.evaluate_property unless internal_sort # this ensures applying converters (e.g. :on_read)
-        table_cells = @table.swt_widget.items.map {|item| @table.column_properties.size.times.map {|i| item.get_text(i)} }
-        model_cells = new_model_collection.to_a.map {|m| @table.cells_for(m)}
-        return if table_cells == model_cells
-        if new_model_collection and new_model_collection.is_a?(Array)
-          @table_items_observer_registration&.unobserve
-          @table_items_observer_registration = observe(new_model_collection, @column_properties)
-          add_dependent(@table_observer_registration => @table_items_observer_registration)
-          @model_collection = new_model_collection
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @model_binding.binding_options[:sync_exec], override_async_exec: @model_binding.binding_options[:async_exec]) do
+          new_model_collection = model_binding_evaluated_property = @model_binding.evaluate_property unless internal_sort # this ensures applying converters (e.g. :on_read)
+          table_cells = @table.swt_widget.items.map {|item| @table.column_properties.size.times.map {|i| item.get_text(i)} }
+          model_cells = new_model_collection.to_a.map {|m| @table.cells_for(m)}
+          return if table_cells == model_cells
+          if new_model_collection and new_model_collection.is_a?(Array)
+            @table_items_observer_registration&.unobserve
+            @table_items_observer_registration = observe(new_model_collection, @column_properties)
+            add_dependent(@table_observer_registration => @table_items_observer_registration)
+            @model_collection = new_model_collection
+          end
+          populate_table(@model_collection, @table, @column_properties, internal_sort: internal_sort)
         end
-        populate_table(@model_collection, @table, @column_properties, internal_sort: internal_sort)
       end
       
       def populate_table(model_collection, parent, column_properties, internal_sort: false)

data/lib/glimmer/data_binding/tree_items_binding.rb

@@ -43,29 +43,33 @@ module Glimmer
         else # assume custom widget
           @tree.body_root.tree_properties = @tree_properties
         end
-        call
-        model = model_binding.base_model
-        observe(model, model_binding.property_name_expression)
-        @tree.on_widget_disposed do |dispose_event|
-          unregister_all_observables
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @model_binding.binding_options[:sync_exec], override_async_exec: @model_binding.binding_options[:async_exec]) do
+          call
+          model = model_binding.base_model
+          observe(model, model_binding.property_name_expression)
+          @tree.on_widget_disposed do |dispose_event|
+            unregister_all_observables
+          end
         end
       end
 
       def call(new_value=nil)
-        @model_tree_root_node = @model_binding.evaluate_property
-        old_tree_items = @tree.all_tree_items
-        old_model_tree_nodes = old_tree_items.map(&:get_data)
-        new_model_tree_nodes = []
-        recursive_depth_first_search(@model_tree_root_node, @tree_properties, new_model_tree_nodes)
-        return if old_model_tree_nodes == new_model_tree_nodes && old_tree_items.map(&:text) == new_model_tree_nodes.map {|model_tree_node| model_tree_node.send(@tree_properties[:text])}
-        populate_tree(@model_tree_root_node, @tree, @tree_properties)
+        Glimmer::SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @model_binding.binding_options[:sync_exec], override_async_exec: @model_binding.binding_options[:async_exec]) do
+          @model_tree_root_node = @model_binding.evaluate_property
+          old_tree_items = @tree.all_tree_items
+          old_model_tree_nodes = old_tree_items.map(&:get_data)
+          new_model_tree_nodes = []
+          recursive_depth_first_search(@model_tree_root_node, @tree_properties, new_model_tree_nodes)
+          return if old_model_tree_nodes == new_model_tree_nodes && old_tree_items.map(&:text) == new_model_tree_nodes.map {|model_tree_node| model_tree_node.send(@tree_properties[:text])}
+          populate_tree(@model_tree_root_node, @tree, @tree_properties)
+        end
       end
 
       def populate_tree(model_tree_root_node, parent, tree_properties)
         # TODO get rid of model_tree_root_node, parent, tree_properties as an argument given it is stored as an instance variable
         # TODO make it change things by delta instead of removing all
         old_tree_items = parent.all_tree_items
-        selected_tree_item_model = parent.swt_widget.getSelection.map(&:get_data).first
+        selected_tree_item_models = parent.swt_widget.getSelection.map(&:get_data)
         old_tree_item_expansion_by_data = old_tree_items.reduce({}) {|hash, ti| hash.merge(ti.getData => ti.getExpanded)}
         old_tree_items.each do |tree_item|
           tree_item.getData('observer_registrations').each(&:unregister)
@@ -74,8 +78,8 @@ module Glimmer
         parent.swt_widget.removeAll
         populate_tree_node(model_tree_root_node, parent.swt_widget, tree_properties)
         parent.all_tree_items.each { |ti| ti.setExpanded(!!old_tree_item_expansion_by_data[ti.getData]) }
-        tree_item_to_select = parent.depth_first_search {|ti| ti.getData == selected_tree_item_model}
-        parent.swt_widget.setSelection(tree_item_to_select)
+        selected_tree_items = parent.depth_first_search {|item| selected_tree_item_models.include?(item.get_data) }
+        parent.swt_widget.setSelection(selected_tree_items)
       end
 
       def populate_tree_node(model_tree_node, parent, tree_properties)

data/lib/glimmer/data_binding/widget_binding.rb

@@ -32,32 +32,30 @@ module Glimmer
       include Observer
 
       attr_reader :widget, :property
-      def initialize(widget, property, translator = nil)
+      def initialize(widget, property, sync_exec: nil, async_exec: nil)
         @widget = widget
         @property = property
-        @translator = translator || proc {|value| value} #TODO check on this it doesn't seem used
-
-        if @widget.respond_to?(:on_widget_disposed)
-          @widget.on_widget_disposed do |dispose_event|
-            unregister_all_observables
+        @sync_exec = sync_exec
+        @async_exec = async_exec
+        SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @sync_exec, override_async_exec: @async_exec) do
+          if @widget.respond_to?(:on_widget_disposed)
+            @widget.on_widget_disposed do |dispose_event|
+              unregister_all_observables
+            end
           end
         end
       end
       
       def call(value)
-        converted_value = translated_value = @translator.call(value)
-      
-        update_operation = lambda do
+        SWT::DisplayProxy.instance.auto_exec(override_sync_exec: @sync_exec, override_async_exec: @async_exec) do
           if @widget.respond_to?(:disposed?) && @widget.disposed?
             unregister_all_observables
             return
           end
-          @widget.set_attribute(@property, converted_value) unless evaluate_property == converted_value
-        end
-        if Config.auto_sync_exec? && Config.require_sync_exec?
-          SWT::DisplayProxy.instance.sync_exec(&update_operation)
-        else
-          update_operation.call
+          # need the rescue false for a scenario with tree items not being equal to model objects raising an exception
+          unless ((value == evaluate_property) rescue false) # need the rescue false for a scenario with tree items not being equal to model objects raising an exception
+            @widget.set_attribute(@property, value)
+          end
         end
       end
       

data/lib/glimmer/dsl/swt/{file_dialog_expression.rb → auto_exec_expression.rb}

@@ -20,28 +20,16 @@
 # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 require 'glimmer/dsl/static_expression'
-require 'glimmer/dsl/parent_expression'
-require 'glimmer/dsl/top_level_expression'
-require 'glimmer/swt/file_dialog_proxy'
+require 'glimmer/dsl/swt/exec_expression'
 
 module Glimmer
   module DSL
     module SWT
-      class FileDialogExpression < StaticExpression
-        include TopLevelExpression
-        include ParentExpression
-        
-        include_package 'org.eclipse.swt.widgets'
-
-        def can_interpret?(parent, keyword, *args, &block)
-          keyword == 'file_dialog' and
-            (parent.nil? or parent.is_a?(Shell) or parent.is_a?(Glimmer::SWT::ShellProxy))
-        end
-  
-        def interpret(parent, keyword, *args, &block)
-          args = [parent] + args unless parent.nil?
-          Glimmer::SWT::FileDialogProxy.send(:new, *args)
-        end
+      # Automatically executes code block against the SWT Event Loop
+      # with sync_exec if needed (inside a thread other than GUI thread)
+      # to manipulate SWT UI objects on the UI thread safely
+      class AutoExecExpression < StaticExpression
+        include ExecExpression
       end
     end
   end