Moved default undo to BitmapOperationTool

PixiEditor/Models/Controllers/BitmapManager.cs

@@ -240,7 +240,6 @@ namespace PixiEditor.Models.Controllers
             {
 
                 cachedHighlight = CoordinatesCalculator.RectangleToCoordinates(0, 0, ToolSize - 1, ToolSize - 1);
-                var center = CoordinatesCalculator.CalculateThicknessCenter(newPosition, ToolSize);
 
                 previewLayerSize = ToolSize;
                 halfSize = (int)Math.Floor(ToolSize / 2f);

PixiEditor/Models/Tools/BitmapOperationTool.cs

@@ -1,10 +1,12 @@
 using System;
 using System.Collections.Generic;
 using System.Windows.Documents;
+using System.Windows.Input;
 using System.Windows.Media;
 using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
+using PixiEditor.Models.Undo;
 using SkiaSharp;
 
 namespace PixiEditor.Models.Tools
@@ -18,6 +20,43 @@ namespace PixiEditor.Models.Tools
         public bool UseDefaultUndoMethod { get; set; } = true;
         public virtual bool UsesShift => true;
 
+        private StorageBasedChange _change;
+
+
         public abstract void Use(Layer layer, List<Coordinates> mouseMove, SKColor color);
+
+        public override void AddUndoProcess(Document document)
+        {
+            var args = new object[] { _change.Document };
+            document.UndoManager.AddUndoChange(_change.ToChange(UndoProcess, args));
+            _change = null;
+        }
+
+        public override void OnRecordingLeftMouseDown(MouseEventArgs e)
+        {
+            if (e.LeftButton != MouseButtonState.Pressed) return;
+
+            Document doc = ViewModels.ViewModelMain.Current.BitmapManager.ActiveDocument;
+            _change = new StorageBasedChange(doc, new[] { doc.ActiveLayer }, true);
+        }
+
+        private void UndoProcess(Layer[] layers, UndoLayer[] data, object[] args)
+        {
+            if (args.Length > 0 && args[0] is Document document)
+            {
+                for (int i = 0; i < layers.Length; i++)
+                {
+                    Layer layer = layers[i];
+                    document.Layers.RemoveAt(data[i].LayerIndex);
+
+                    document.Layers.Insert(data[i].LayerIndex, layer);
+                    if (data[i].IsActive)
+                    {
+                        document.SetMainActiveLayer(data[i].LayerIndex);
+                    }
+                }
+
+            }
+        }
     }
 }

PixiEditor/Models/Tools/Tool.cs

@@ -55,7 +55,6 @@ namespace PixiEditor.Models.Tools
 
         private bool isActive;
         private string actionDisplay = string.Empty;
-        private StorageBasedChange _change;
 
         public virtual void OnMouseDown(MouseEventArgs e)
         {
@@ -63,28 +62,6 @@ namespace PixiEditor.Models.Tools
 
         public virtual void AddUndoProcess(Document document)
         {
-            var args = new object[] { _change.Document };
-            document.UndoManager.AddUndoChange(_change.ToChange(UndoProcess, args));
-            _change = null;
-        }
-
-        private void UndoProcess(Layer[] layers, UndoLayer[] data, object[] args)
-        {
-            if(args.Length > 0 && args[0] is Document document)
-            {
-                for (int i = 0; i < layers.Length; i++)
-                {
-                    Layer layer = layers[i];
-                    document.Layers.RemoveAt(data[i].LayerIndex);
-
-                    document.Layers.Insert(data[i].LayerIndex, layer);
-                    if (data[i].IsActive)
-                    {
-                        document.SetMainActiveLayer(data[i].LayerIndex);
-                    }
-                }
-
-            }
         }
 
         public virtual void OnMouseUp(MouseEventArgs e)
@@ -105,10 +82,6 @@ namespace PixiEditor.Models.Tools
 
         public virtual void OnRecordingLeftMouseDown(MouseEventArgs e)
         {
-            if (e.LeftButton != MouseButtonState.Pressed) return;
-
-            Document doc = ViewModels.ViewModelMain.Current.BitmapManager.ActiveDocument;
-            _change = new StorageBasedChange(doc, new[] { doc.ActiveLayer }, true);
         }
 
         public virtual void OnStoppedRecordingMouseUp(MouseEventArgs e)