glimmer-cs-gladiator 0.5.2 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5de57f9122b239134e9567e99976ed7c5e859edf89a91f72f15f257597ee0815
-  data.tar.gz: 99414159618bea5a0a99fd37772424b452f26471ca6e5dd791ff0ce26135646c
+  metadata.gz: 3c07882b7fc3a12df1d64f02148ac6d018c1edd45fdea10c6d5285bdd55df137
+  data.tar.gz: fdacfe1e7f81879c96f0d746cc1e62ec66e6a202a934dae2f9bb7c1027f91a95
 SHA512:
-  metadata.gz: 6e7d7c9cca67e351dbf169d785c6ef927bcd911f03a5b4505fdd2973c9b9291b1a72f8475dceeae592bb3d56de85a65f96b9f1cb96575dc91768d13b46f52749
-  data.tar.gz: 35c3dc09a4f5d0e29d8e52dd0cbae1ffc879f6d7cd3d5ec0cc2cbabde3475f3898a959c34d7adf60789c9a8c8fa6e2e5ebcfefe9e5e1db8ba0e7281b268ef572
+  metadata.gz: 42e38a486000c47409ecefd8f120950e856abd9163a3131b8733e65f9bbb337437ed660374c363ccded21db8c4491c158922be6e5510b2af2a8ed2b3e5aa96f1
+  data.tar.gz: 36fcaddd1f4d6766f53fb19bd26aaeeaf5e2f86164e6244620f3d5cc5e9067bf48b2172005c850d408d1dc7a80c43dabbe5e9f59c6be53664de4f312d78de844

data/README.md

@@ -1,4 +1,4 @@
-# <img src='https://raw.githubusercontent.com/AndyObtiva/glimmer-cs-gladiator/master/images/glimmer-cs-gladiator-logo.svg' height=85 /> Gladiator 0.5.2 - [Ugliest Text Editor Ever](https://www.reddit.com/r/ruby/comments/hgve8k/gladiator_glimmer_editor_ugliest_text_editor_ever/)
+# <img src='https://raw.githubusercontent.com/AndyObtiva/glimmer-cs-gladiator/master/images/glimmer-cs-gladiator-logo.svg' height=85 /> Gladiator 0.6.2 - [Ugliest Text Editor Ever](https://www.reddit.com/r/ruby/comments/hgve8k/gladiator_glimmer_editor_ugliest_text_editor_ever/)
 ## [<img src="https://raw.githubusercontent.com/AndyObtiva/glimmer/master/images/glimmer-logo-hi-res.png" height=40 /> Glimmer Custom Shell](https://github.com/AndyObtiva/glimmer-dsl-swt#custom-shell-gem)
 [![Gem Version](https://badge.fury.io/rb/glimmer-cs-gladiator.svg)](http://badge.fury.io/rb/glimmer-cs-gladiator)
 
@@ -11,18 +11,14 @@ I leave building truly professional text editors to software tooling experts who
 
 ## Features
 
-Gladiator currently supports the following text editing features (including keyboard shortcuts with Mac CMD=CTRL on Windows/Linux):
-- Ruby Syntax Highlighting with colors
-- File explorer navigation with context menu to open file, rename, delete, add new file, add new directory, or refresh tree (CMD+T)
-- File lookup by name ignoring slashes, underscores, and dots to ease lookup (CMD+R)
-- Watch open file for external changes to automatically refresh in editor
-- Watch project subdirectories for changes to automatically refresh in file explorer/file lookup
-- Find & Replace (CMD+F)
+Gladiator currently supports the following text editing features (including keyboard shortcuts with Mac CMD=CTRL on Windows/Linux)
+
+### Text Editor
+
+- Text Editor with Colored Ruby Syntax Highlighting
 - Show Line Numbers
-- Jump to Line (CMD+L)
-- Multiple tab support (CMD+SHIFT+[ & CMD+SHIFT+] for tab navigation. CMD+1-9 to jump to a specific tab)
-- Remember opened tabs, caret position, top line, window size, and window location
-- Autosave on focus out/quit/open new file
+- Multi-tab support (CMD+SHIFT+[ & CMD+SHIFT+] for tab navigation. CMD+1-9 to jump to a specific tab)
+- Drag and drop split pane (drag a file from File Tree or File Lookup List, and it splits the pane)
 - Duplicate Line(s)/selection (CMD+D)
 - Kill Line(s)/selection (CMD+K)
 - Move line/selection up (CMD+UP)
@@ -30,10 +26,41 @@ Gladiator currently supports the following text editing features (including keyb
 - Comment/Uncomment line/selection (CMD+/)
 - Indent/Unindent line/selection (CMD+] & CMD+[)
 - Insert/Prefix New Line (CMD+ENTER & CMD+SHIFT+ENTER)
-- Drag and Drop Text Editor Split Screen (drag a file from File Tree or File Lookup List, and it splits the screen)
-- Run Ruby code (CMD+SHIFT+R)
-- Scratchpad for running arbitrary Ruby/Glimmer code without saving to disk (CMD+SHIFT+S)
-- Change Split Orientation to Horizontal/Vertical via View Menu (CMD+SHIFT+O)
+
+### File Explorer Tree
+
+- Collapsable file explorer tree listing files and directories for open project
+- Context menu to open file, rename, delete, add new file, add new directory, and refresh tree
+- Jump to open file in tree (CMD+T)
+
+### File Lookup List Filter
+
+- Collapsable file lookup list filter (CMD+R)
+- Semi-fuzzy filtering by ignoring slashes, underscores, and dots to ease lookup
+
+### Navigation Area
+
+- Show current text editor file name
+- Show file navigation stats (Caret Position / Line Position / Selection Count / Top Pixel)
+- Jump to Line (CMD+L)
+- Find & Replace (CMD+F)
+
+### Menus
+
+- File Menu to Open Project (CMD+SHIFT+P) and open Scratchpad for running arbitrary Ruby/Glimmer code without saving to disk (CMD+SHIFT+S)
+- View Menu to Split Pane & Change Split Orientation to Horizontal/Vertical (CMD+SHIFT+O)
+- Run Menu to run Ruby code (CMD+SHIFT+R)
+
+### Watch External Changes
+
+- Watch open file for external changes to automatically refresh in editor
+- Watch project subdirectories for changes to automatically refresh in file explorer/file lookup
+
+### Automatic Data Management
+
+- Autosave on focus out/quit/open new file
+- Remember opened tabs, caret position, top pixel, window size, and window location
+- [Default](#configuration) "ignore paths" to avoid bogging down editor with irrelevant directory files
 
 ## Platforms
 
@@ -43,7 +70,7 @@ Gladiator currently supports the following text editing features (including keyb
 
 ## Pre-requisites
 
-- [Glimmer DSL for SWT](https://github.com/AndyObtiva/glimmer-dsl-swt) (JRuby Desktop Development GUI Library): '>= 4.17.2.0', '< 5.0.0.0' (dependency included in Ruby gem).
+- [Glimmer DSL for SWT](https://github.com/AndyObtiva/glimmer-dsl-swt) (JRuby Desktop Development GUI Framework): '>= 4.17.2.0', '< 5.0.0.0' (dependency included in Ruby gem).
 - [JRuby](https://www.jruby.org/download): Same version required by [Glimmer](https://github.com/AndyObtiva/glimmer-dsl-swt)
 - [JDK](https://www.oracle.com/java/technologies/javase-downloads.html): Same version required by [Glimmer](https://github.com/AndyObtiva/glimmer-dsl-swt)
 
@@ -77,6 +104,12 @@ You may run the `gladiator` command to bring up the text editor in the project d
 gladiator
 ```
 
+On Linux, you may need to run with extra memory via this command instead:
+
+```
+gladiator -J-Xmx1200M
+```
+
 On Windows, you may need to run with extra memory via this command instead:
 
 ```
@@ -103,7 +136,7 @@ To reuse Gladiator as a Glimmer Custom Shell inside another Glimmer application,
 following to the application's `Gemfile`:
 
 ```
-gem 'glimmer-cs-gladiator', '~> 0.5.2'
+gem 'glimmer-cs-gladiator', '~> 0.6.2'
 ```
 
 Run:
@@ -129,16 +162,12 @@ Opens Gladiator with "/Users/User/code" as the root directory.
 
 ## Configuration
 
-Gladiator automatically saves configuration data in a `.gladiator` file at the directory it is run from.
+Gladiator automatically saves configuration data in a `.gladiator` file at the directory it is run from. It may be edited to add extra ignore paths.
 
 It currently remembers:
-- Last opened file
-- Caret position
-- Top line position
-- Window size
-- Opened tabs
-- Split tabs
-- Ignore Paths
+- Last opened files (in both split panes if split)
+- Window size and position
+- Ignore Paths (default: '.gladiator', '.git', 'coverage', 'packages', 'node_modules', 'tmp', 'vendor')
 
 ## Gotcha
 
@@ -171,6 +200,6 @@ Copyright (c) 2020 Andy Maleh. See [LICENSE.txt](LICENSE.txt) for further detail
 
 --
 
-[<img src="https://raw.githubusercontent.com/AndyObtiva/glimmer/master/images/glimmer-logo-hi-res.png" height=40 />](https://github.com/AndyObtiva/glimmer) Built with [Glimmer DSL for SWT](https://github.com/AndyObtiva/glimmer-dsl-swt) (JRuby Desktop Development GUI Library)
+[<img src="https://raw.githubusercontent.com/AndyObtiva/glimmer/master/images/glimmer-logo-hi-res.png" height=40 />](https://github.com/AndyObtiva/glimmer) Built with [Glimmer DSL for SWT](https://github.com/AndyObtiva/glimmer-dsl-swt) (JRuby Desktop Development GUI Framework)
 
 Gladiator icon made by <a href="https://www.flaticon.com/authors/freepik" title="Freepik">Freepik</a> from <a href="https://www.flaticon.com/" title="Flaticon">www.flaticon.com</a>

data/VERSION

@@ -1 +1 @@
-0.5.2
+0.6.2

data/lib/models/glimmer/gladiator/command.rb

@@ -1,6 +1,8 @@
 module Glimmer
   class Gladiator
     class Command
+      include Glimmer
+      
       class << self
         include Glimmer
         
@@ -13,11 +15,18 @@ module Glimmer
           command_history[file] ||= [Command.new(file)]
         end
         
-        def do(file, method = nil, command: nil)
-          command ||= Command.new(file, method)
-          command_history_for(file)&.last&.next_command = command
-          command.do
-          command_history_for(file) << command
+        def do(file, method = nil, *args, command: nil)
+          if command.nil?
+            command ||= Command.new(file, method, *args)
+            command.previous_command = command_history_for(file).last
+            unless command_history_for(file).last.method == :change_content! && method == :change_content!
+              command_history_for(file).last.next_command = command
+            end
+            command.do
+            command_history_for(file) << command unless command_history_for(file).last.method == :change_content! && method == :change_content!          
+          else
+            command_history_for(file) << command
+          end
         end
         
         def undo(file)
@@ -30,15 +39,22 @@ module Glimmer
           command = command_history_for(file).last
           command&.redo
         end
+        
+        def clear(file)
+          command_history[file] = [Command.new(file)]
+        end
       end
     
-      attr_accessor :file, :method, :next_command, :previous_file_content, :previous_file_caret_position, :previous_file_selection_count
+      attr_accessor :file, :method, :args, :previous_command, :next_command, 
+                    :file_dirty_content, :file_caret_position, :file_selection_count, :previous_file_dirty_content, :previous_file_caret_position, :previous_file_selection_count
     
-      def initialize(file, method = nil)
+      def initialize(file, method = nil, *args)
         @file = file
         @method = method
+        @args = args
       end
       
+      
       def native?
         @method.nil?
       end
@@ -56,6 +72,9 @@ module Glimmer
       
       def redo
         return if next_command.nil?# || next_command.native?
+        @file.dirty_content = next_command.file_dirty_content.clone
+        @file.caret_position = next_command.file_caret_position
+        @file.selection_count = next_command.file_selection_count
         Command.do(next_command.file, command: next_command)
       end
       
@@ -63,6 +82,10 @@ module Glimmer
         @previous_file_dirty_content = @file.dirty_content.clone
         @previous_file_caret_position = @file.caret_position
         @previous_file_selection_count = @file.selection_count
+        if @method == :change_content!
+          @previous_file_caret_position = @file.last_caret_position
+          @previous_file_selection_count = @file.last_selection_count
+        end
       end
       
       def restore
@@ -72,7 +95,17 @@ module Glimmer
       end
       
       def execute
-        @file.send(@method)
+        @file.start_command
+        @file.send(@method, *@args)
+        @file.end_command
+        @file_dirty_content = @file.dirty_content.clone
+        @file_caret_position = @file.caret_position
+        @file_selection_count = @file.selection_count
+        if previous_command.method == :change_content! && @method == :change_content!
+          previous_command.file_dirty_content = @file_dirty_content
+          previous_command.file_caret_position = @file_caret_position
+          previous_command.file_selection_count = @file_selection_count
+        end
       end
     end
   end

data/lib/models/glimmer/gladiator/dir.rb

@@ -6,8 +6,6 @@ module Glimmer
       include Glimmer
       include Glimmer::DataBinding::ObservableModel
 
-      REFRESH_DELAY = 7
-  
       attr_accessor :selected_child, :filter, :children, :filtered_path_options, :filtered_path, :display_path, :ignore_paths
       attr_reader :name, :parent, :path
       attr_writer :all_children
@@ -15,19 +13,30 @@ module Glimmer
       def initialize(path, project_dir = nil)
         @project_dir = project_dir
         if is_local_dir
-         @filewatcher = Filewatcher.new(path)
-         @thread = Thread.new(@filewatcher) do |fw|
-           fw.watch do |filename, event|
-             if @last_update.nil? || (Time.now.to_f - @last_update) > REFRESH_DELAY
-               refresh if !filename.include?('new_file') && !selected_child_path_history.include?(filename) && filename != selected_child_path
-             end
-             @last_update = Time.now.to_f
-           end
-         end
+          @filewatcher = Filewatcher.new(path)
+          Thread.new(@filewatcher) do |fw|
+            begin
+              fw.watch do |filename, event|
+                # TODO do fine grained processing of events for enhanced performance (e.g. dir refresh vs file change)
+                # TODO do fine grained file change only without a refresh delay for enhanced performance
+                begin
+                  if !@refresh_in_progress && !filename.include?('new_file') && (event != :updated || find_child_file(filename).nil?)
+                    Thread.new {
+                      refresh
+                    }
+                  end
+                rescue => e
+                  puts e.full_message
+                end
+              end
+            rescue => e
+              puts e.full_message
+            end
+          end
         end
         self.path = ::File.expand_path(path)
         @name = ::File.basename(::File.expand_path(path))
-        @ignore_paths = ['.gladiator', '.git', 'coverage', 'packages', 'tmp', 'vendor']
+        @ignore_paths = ['.gladiator', '.git', 'coverage', 'packages', 'node_modules', 'tmp', 'vendor']
         self.filtered_path_options = []
       end
       
@@ -67,7 +76,7 @@ module Glimmer
             result || p.include?(ignore_path)
           end
         end.map do |p|
-          ::File.file?(p) ? Gladiator::File.new(p, project_dir) : Gladiator::Dir.new(p, project_dir)
+          ::File.file?(p) ? File.new(p, project_dir) : Dir.new(p, project_dir)
         end.sort_by do |c|
           c.path.to_s.downcase
         end.sort_by do |c|
@@ -76,6 +85,22 @@ module Glimmer
           child.retrieve_children if child.is_a?(Dir)
         end
       end
+      
+      def find_child_file(child_path)
+        depth_first_search_file(self, child_path)
+      end
+      
+      def depth_first_search_file(dir, file_path)
+        dir.children.each do |child|
+          if child.is_a?(File)
+            return child if child.path.include?(file_path)
+          else
+            result = depth_first_search_file(child, file_path)
+            return result unless result.nil?
+          end
+        end
+        nil
+      end
   
       def selected_child_path_history
         @selected_child_path_history ||= []
@@ -91,6 +116,7 @@ module Glimmer
 
       def refresh(async: true, force: false)
         return if @refresh_paused && !force
+        @refresh_in_progress = true
         retrieve_children
         collect_all_children
         refresh_operation = lambda do
@@ -102,6 +128,7 @@ module Glimmer
         else
           sync_exec(&refresh_operation)
         end
+        @refresh_in_progress = false
       end
   
       def filter=(value)
@@ -144,7 +171,7 @@ module Glimmer
       def selected_child_path=(selected_path)
         return (project_dir.selected_child = nil) if selected_path.nil?
         # scratchpad scenario
-        if selected_path.empty? #scratchpad
+        if selected_path.empty? # Scratchpad
           @selected_child&.write_dirty_content
           return (self.selected_child = File.new)
         end
@@ -154,7 +181,7 @@ module Glimmer
         selected_path = full_selected_path
         if ::File.file?(selected_path)
           @selected_child&.write_dirty_content
-          new_child = Gladiator::File.new(selected_path, project_dir)
+          new_child = find_child_file(selected_path)
           begin
             unless new_child.dirty_content.nil?
               self.selected_child&.stop_filewatcher
@@ -175,6 +202,7 @@ module Glimmer
       end
       
       def selected_child=(new_child)
+        return if selected_child == new_child
         file_properties = @selected_child&.backup_properties if @selected_child == new_child
         @selected_child = new_child
         @selected_child.restore_properties(file_properties) if file_properties
@@ -191,7 +219,7 @@ module Glimmer
       def eql?(other)
         self.path.eql?(other&.path)
       end
-      
+
       def hash
         self.path.hash
       end

data/lib/models/glimmer/gladiator/file.rb

@@ -3,7 +3,7 @@ module Glimmer
     class File
       include Glimmer
 
-      attr_accessor :dirty_content, :line_numbers_content, :selection, :line_number, :find_text, :replace_text, :top_pixel, :display_path, :case_sensitive
+      attr_accessor :line_numbers_content, :line_number, :find_text, :replace_text, :top_pixel, :display_path, :case_sensitive, :caret_position, :selection_count, :last_caret_position, :last_selection_count, :line_position
       attr_reader :name, :path, :project_dir
 
       def initialize(path='', project_dir=nil)
@@ -13,33 +13,55 @@ module Glimmer
         @name = path.empty? ? 'Scratchpad' : ::File.basename(path)
         self.path = ::File.expand_path(path) unless path.empty?
         @top_pixel = 0
+        @caret_position = 0
         @selection_count = 0
-        @selection = Point.new(0, 0 + @selection_count)
-        read_dirty_content = path.empty? ? '' : ::File.read(path)
-        begin
-          # test read dirty content
-          read_dirty_content.split("\n")
-          observe(self, :dirty_content) do
-            lines_text_size = lines.size.to_s.size
-            self.line_numbers_content = lines.size.times.map {|n| (' ' * (lines_text_size - (n+1).to_s.size)) + (n+1).to_s }.join("\n")
-          end
-          @line_number = 1
-          self.dirty_content = read_dirty_content
-          observe(self, :selection) do
-            self.line_number = line_index_for_caret_position(caret_position) + 1
-          end
-          observe(self, :line_number) do
-            if line_number
+        @last_selection_count = 0
+        @line_number = 1
+        @init = nil
+      end
+      
+      def init_content
+        unless @init
+          @init = true
+          begin
+            # test read dirty content
+            observe(self, :dirty_content) do
+              line_count = lines.empty? ? 1 : lines.size
+              lines_text_size = [line_count.to_s.size, 4].max
+              old_top_pixel = top_pixel
+              self.line_numbers_content = line_count.times.map {|n| (' ' * (lines_text_size - (n+1).to_s.size)) + (n+1).to_s }.join("\n")
+              self.top_pixel = old_top_pixel
+            end
+            the_dirty_content = read_dirty_content
+            the_dirty_content.split("\n") # test that it is not a binary file (crashes to rescue block otherwise)
+            self.dirty_content = the_dirty_content
+            observe(self, :caret_position) do |new_caret_position|
+              update_line_number_from_caret_position(new_caret_position)
+            end
+            observe(self, :line_number) do |new_line_number|
               line_index = line_number - 1
               new_caret_position = caret_position_for_line_index(line_index)
-              self.caret_position = new_caret_position unless self.caret_position && line_index_for_caret_position(new_caret_position) == line_index_for_caret_position(caret_position)
+              current_caret_position = caret_position
+              line_index_for_new_caret_position = line_index_for_caret_position(new_caret_position)
+              line_index_for_current_caret_position = line_index_for_caret_position(current_caret_position)
+              self.caret_position = new_caret_position unless (current_caret_position && line_index_for_new_caret_position == line_index_for_current_caret_position)
             end
+          rescue # in case of a binary file
+            stop_filewatcher
           end
-        rescue
-          # no op in case of a binary file
         end
       end
-
+      
+      def update_line_number_from_caret_position(new_caret_position)
+        new_line_number = line_index_for_caret_position(caret_position) + 1
+        current_line_number = line_number
+        unless (current_line_number && current_line_number == new_line_number)
+          self.line_number = new_line_number
+          # TODO check if the following line is needed
+          self.line_position = caret_position - caret_position_for_line_index(line_number - 1) + 1
+        end
+      end
+      
       def path=(the_path)
         @path = the_path
         generate_display_path
@@ -49,6 +71,15 @@ module Glimmer
         return if @path.empty?
         @display_path = @path.sub(project_dir.path, '').sub(/^\//, '')
       end
+
+      def name=(the_name)
+        new_path = path.sub(/#{Regexp.escape(@name)}$/, the_name) unless scratchpad?
+        @name = the_name
+        if !scratchpad? && ::File.exist?(path)
+          FileUtils.mv(path, new_path)
+          self.path = new_path
+        end
+      end
       
       def scratchpad?
         path.to_s.empty?
@@ -66,50 +97,74 @@ module Glimmer
           send("#{property}=", value)
         end
       end
+      
+      def caret_position=(value)
+        @last_caret_position = @caret_position
+        @caret_position = value
+      end
 
-      # to use for widget data-binding
-      def content=(value)
-        value = value.gsub("\t", '  ')
-        if dirty_content != value
-          Command.do(self) # record a native (OS-widget) operation
-          self.dirty_content = value
-        end
+      def selection_count=(value)
+        #@last_selection_count = @selection_count
+        @selection_count = value
+        @last_selection_count = @selection_count
       end
 
+      def dirty_content
+        init_content
+        @dirty_content
+      end
+      
+      def dirty_content=(the_content)
+        # TODO set partial dirty content by line(s) for enhanced performance
+        @dirty_content = the_content
+        old_caret_position = caret_position
+        old_top_pixel = top_pixel
+        
+        notify_observers(:content)
+        if @formatting_dirty_content_for_writing
+          self.caret_position = old_caret_position
+          self.top_pixel = old_top_pixel
+        end
+      end
+      
       def content
         dirty_content
       end
 
-      def caret_position=(value)
-        old_top_pixel = top_pixel
-        self.selection = Point.new(value, value + selection_count.to_i)
-        self.top_pixel = old_top_pixel
+      # to use for widget data-binding
+      def content=(value)
+        value = value.gsub("\t", '  ')
+        if dirty_content != value
+          Command.do(self, :change_content!, value)
+        end
       end
-
-      def caret_position
-        selection.x
+      
+      def change_content!(value)
+        self.dirty_content = value
+        update_line_number_from_caret_position(caret_position)
       end
 
-      def selection_count
-        selection.y - selection.x
+      def start_command
+        @commmand_in_progress = true
       end
-
-      def selection_count=(value)
-        self.selection = Point.new(caret_position, caret_position + value.to_i)
+      
+      def end_command
+        @commmand_in_progress = false
       end
-
-      def name=(the_name)
-        new_path = path.sub(/#{Regexp.escape(@name)}$/, the_name) unless scratchpad?
-        @name = the_name
-        if !scratchpad? && ::File.exist?(path)
-          FileUtils.mv(path, new_path)
-          self.path = new_path
-        end
+      
+      def command_in_progress?
+        @commmand_in_progress
       end
       
-      def dirty_content=(the_content)
-        @dirty_content = the_content
-        notify_observers(:content)
+      def close
+        stop_filewatcher
+        remove_all_observers
+        initialize(path, project_dir)
+        Command.clear(self)
+      end
+      
+      def read_dirty_content
+        path.empty? ? '' : ::File.read(path)
       end
 
       def start_filewatcher
@@ -117,15 +172,13 @@ module Glimmer
         @filewatcher = Filewatcher.new(@path)
         @thread = Thread.new(@filewatcher) do |fw|
           fw.watch do |filename, event|
-            begin
-              read_dirty_content = ::File.read(path)
-              # test read dirty content
-              read_dirty_content.split("\n")
-              async_exec do
+            async_exec do
+              begin
                 self.dirty_content = read_dirty_content if read_dirty_content != dirty_content
+              rescue StandardError, Errno::ENOENT
+                # in case of a binary file
+                stop_filewatcher
               end
-            rescue
-              # no op in case of a binary file
             end
           end
         end
@@ -135,21 +188,28 @@ module Glimmer
         @filewatcher&.stop
       end
 
-      def format_dirty_content_for_writing!
-        new_dirty_content = dirty_content.split("\n").map {|line| line.strip.empty? ? line : line.rstrip }.join("\n")
-        new_dirty_content = "#{new_dirty_content.gsub("\r\n", "\n").gsub("\r", "\n").sub(/\n+\z/, '')}\n"
-        self.dirty_content = new_dirty_content if new_dirty_content != self.dirty_content
-      end
-
       def write_dirty_content
-        return if scratchpad? || !::File.exist?(path)
+        # TODO write partial dirty content by line(s) for enhanced performance
+        return if scratchpad? || !::File.exist?(path) || !::File.exists?(path) || read_dirty_content == dirty_content
         format_dirty_content_for_writing!
-        ::File.write(path, dirty_content) if ::File.exists?(path)
-      rescue => e
+        ::File.write(path, dirty_content)
+      rescue StandardError, ArgumentError => e
         puts "Error in writing dirty content for #{path}"
         puts e.full_message
       end
 
+      def format_dirty_content_for_writing!
+        return if @commmand_in_progress
+        # TODO  f ix c ar e t pos it ion after formatting dirty content (diff?)
+        new_dirty_content = dirty_content.to_s.split("\n").map {|line| line.strip.empty? ? line : line.rstrip }.join("\n")
+        new_dirty_content = "#{new_dirty_content.gsub("\r\n", "\n").gsub("\r", "\n").sub(/\n+\z/, '')}\n"
+        if new_dirty_content != self.dirty_content
+          @formatting_dirty_content_for_writing = true
+          self.dirty_content = new_dirty_content
+          @formatting_dirty_content_for_writing = false
+        end
+      end
+
       def write_raw_dirty_content
         return if scratchpad? || !::File.exist?(path)
         ::File.write(path, dirty_content) if ::File.exists?(path)
@@ -185,7 +245,7 @@ module Glimmer
         self.caret_position = caret_position_for_line_index(line_number) + current_line_indentation.size
         self.selection_count = 0
       end
-
+      
       def comment_line!
         old_lines = lines
         return if old_lines.size < 1
@@ -219,6 +279,7 @@ module Glimmer
           new_caret_position = old_caret_position + delta
           new_caret_position = [new_caret_position, old_caret_position_line_caret_position].max
           self.caret_position = new_caret_position
+          self.selection_count = 0
         end
       end
 
@@ -243,6 +304,7 @@ module Glimmer
           self.selection_count = (caret_position_for_line_index(old_end_caret_line_index + 1) - self.caret_position)
         else
           self.caret_position = old_caret_position + delta
+          self.selection_count = 0
         end
       end
 
@@ -274,6 +336,7 @@ module Glimmer
           new_caret_position = old_caret_position + delta
           new_caret_position = [new_caret_position, old_caret_position_line_caret_position].max
           self.caret_position = new_caret_position
+          self.selection_count = 0
         end
       end
 
@@ -311,6 +374,7 @@ module Glimmer
           self.selection_count = (caret_position_for_line_index(old_end_caret_line_index + 1) - self.caret_position)
         else
           self.caret_position = old_caret_position + delta
+          self.selection_count = 0
         end
       end
 
@@ -471,7 +535,7 @@ module Glimmer
       def line_index_for_caret_position(caret_position)
         dirty_content[0...caret_position.to_i].count("\n")
       end
-
+      
       def caret_position_for_line_index(line_index)
         cp = lines[0...line_index].join("\n").size
         cp += 1 if line_index > 0