Moved Value field to Setting<T> and used generic settings in the application

PixiEditor/Models/Controllers/BitmapManager.cs

@@ -12,8 +12,6 @@ using PixiEditor.Models.ImageManipulation;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Tools;
-using PixiEditor.Models.Tools.ToolSettings;
-using PixiEditor.Models.Tools.ToolSettings.Settings;
 
 namespace PixiEditor.Models.Controllers
 {
@@ -47,12 +45,12 @@ namespace PixiEditor.Models.Controllers
 
         public int ToolSize
         {
-            get => SelectedTool.Toolbar.GetSetting("ToolSize") != null
-            ? (int)SelectedTool.Toolbar.GetSetting("ToolSize").Value
+            get => SelectedTool.Toolbar.GetSetting<int>("ToolSize") != null
+            ? SelectedTool.Toolbar.GetSetting<int>("ToolSize").Value
             : 1;
             set
             {
-                if (SelectedTool.Toolbar.GetSetting("ToolSize") is Setting toolSize)
+                if (SelectedTool.Toolbar.GetSetting<int>("ToolSize") is var toolSize)
                 {
                     toolSize.Value = value;
                     HighlightPixels(MousePositionConverter.CurrentCoordinates);

PixiEditor/Models/Tools/ToolSettings/Settings/BoolSetting.cs

@@ -5,7 +5,7 @@ using System.Windows.Data;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public class BoolSetting : Setting
+    public class BoolSetting : Setting<bool>
     {
         public BoolSetting(string name, string label = "") : base(name)
         {
@@ -25,7 +25,7 @@ namespace PixiEditor.Models.Tools.ToolSettings.Settings
         {
             CheckBox checkBox = new CheckBox
             {
-                IsChecked = (bool) Value,
+                IsChecked = Value,
                 VerticalAlignment = VerticalAlignment.Center
             };
 

PixiEditor/Models/Tools/ToolSettings/Settings/ColorSetting.cs

@@ -4,7 +4,7 @@ using ColorPicker;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public class ColorSetting : Setting
+    public class ColorSetting : Setting<Color>
     {
         public ColorSetting(string name, string label = "") : base(name)
         {

PixiEditor/Models/Tools/ToolSettings/Settings/DropdownSetting.cs

@@ -5,7 +5,7 @@ using System.Windows.Data;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public class DropdownSetting : Setting
+    public class DropdownSetting : Setting<object>
     {
         public string[] Values { get; set; }
 

PixiEditor/Models/Tools/ToolSettings/Settings/FloatSetting.cs

@@ -3,7 +3,7 @@ using PixiEditor.Views;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public class FloatSetting : Setting
+    public class FloatSetting : Setting<float>
     {
         public float Min { get; set; }
         public float Max { get; set; }

PixiEditor/Models/Tools/ToolSettings/Settings/Setting.cs

@@ -3,13 +3,14 @@ using PixiEditor.Helpers;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public abstract class Setting<T> : NotifyableObject
+    public abstract class Setting<T> : Setting
     {
-        public string Name { get; }
-
-        public string Label { get; set; }
+        private T value;
 
-        public bool HasLabel => !string.IsNullOrEmpty(Label);
+        protected Setting(string name)
+            : base(name)
+        {
+        }
 
         public T Value
         {
@@ -20,22 +21,21 @@ namespace PixiEditor.Models.Tools.ToolSettings.Settings
                 RaisePropertyChanged("Value");
             }
         }
+    }
 
-        public Control SettingControl { get; set; }
+    public abstract class Setting : NotifyableObject
+    {
+        public string Name { get; }
 
-        private T value;
+        public string Label { get; set; }
 
-        protected Setting(string name)
-        {
-            Name = name;
-        }
-    }
+        public bool HasLabel => !string.IsNullOrEmpty(Label);
+
+        public Control SettingControl { get; set; }
 
-    public abstract class Setting : Setting<object>
-    {
         protected Setting(string name)
-            : base(name)
         {
+            Name = name;
         }
     }
 }

PixiEditor/Models/Tools/ToolSettings/Settings/SizeSetting.cs

@@ -7,7 +7,7 @@ using PixiEditor.Helpers.Behaviours;
 
 namespace PixiEditor.Models.Tools.ToolSettings.Settings
 {
-    public class SizeSetting : Setting
+    public class SizeSetting : Setting<int>
     {
         public SizeSetting(string name, string label = null) : base(name)
         {
@@ -28,7 +28,7 @@ namespace PixiEditor.Models.Tools.ToolSettings.Settings
 
             if (Application.Current != null)
             {
-                tb.Style = (Style)Application.Current.TryFindResource("DarkTextBoxStyle"); ;
+                tb.Style = (Style)Application.Current.TryFindResource("DarkTextBoxStyle");
             }
 
             Binding binding = new Binding("Value")

PixiEditor/Models/Tools/ToolSettings/Toolbars/Toolbar.cs

@@ -21,16 +21,34 @@ namespace PixiEditor.Models.Tools.ToolSettings.Toolbars
             return Settings.FirstOrDefault(x => string.Equals(x.Name, name, StringComparison.CurrentCultureIgnoreCase));
         }
 
+        /// <summary>
+        ///     Gets setting with given type T in toolbar by name.
+        /// </summary>
+        /// <param name="name">Setting name, non case sensitive</param>
+        /// <returns></returns>
+        public Setting<T> GetSetting<T>(string name)
+        {
+            Setting setting =  Settings.FirstOrDefault(currentSetting => string.Equals(currentSetting.Name, name, StringComparison.CurrentCultureIgnoreCase));
+
+            if (setting == null)
+                return null;
+
+            if (setting is Setting<T> convertedSetting)
+                return convertedSetting;
+
+            throw new Exception("Setting has value with unexpected type");
+        }
+
         /// <summary>
         ///     Saves current toolbar state, so other toolbars with common settings can load them.
         /// </summary>
         public void SaveToolbarSettings()
         {
-            for (int i = 0; i < Settings.Count; i++)
-                if (SharedSettings.Any(x => x.Name == Settings[i].Name))
-                    SharedSettings.First(x => x.Name == Settings[i].Name).Value = Settings[i].Value;
-                else
-                    SharedSettings.Add(Settings[i]);
+            SharedSettings.Clear();
+            foreach (Setting setting in Settings)
+            {
+                SharedSettings.Add(setting);
+            }
         }
 
         /// <summary>
@@ -38,9 +56,11 @@ namespace PixiEditor.Models.Tools.ToolSettings.Toolbars
         /// </summary>
         public void LoadSharedSettings()
         {
-            for (int i = 0; i < SharedSettings.Count; i++)
-                if (Settings.Any(x => x.Name == SharedSettings[i].Name))
-                    Settings.First(x => x.Name == SharedSettings[i].Name).Value = SharedSettings[i].Value;
+            Settings.Clear();
+            foreach (Setting setting in SharedSettings)
+            {
+                Settings.Add(setting);
+            }
         }
     }
 }

PixiEditor/Models/Tools/Tools/BrightnessTool.cs

@@ -3,7 +3,6 @@ using System.Collections.Generic;
 using System.Windows.Controls;
 using System.Windows.Input;
 using System.Windows.Media;
-using System.Windows.Media.Imaging;
 using PixiEditor.Models.Colors;
 using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.Enums;
@@ -35,9 +34,9 @@ namespace PixiEditor.Models.Tools.Tools
 
         public override LayerChange[] Use(Layer layer, Coordinates[] coordinates, Color color)
         {
-            int toolSize = (int) Toolbar.GetSetting("ToolSize").Value;
-            float correctionFactor = (float) Toolbar.GetSetting("CorrectionFactor").Value;
-            Enum.TryParse<BrightnessMode>((Toolbar.GetSetting("Mode").Value as ComboBoxItem)?.Content as string, out var mode);
+            int toolSize = Toolbar.GetSetting<int>("ToolSize").Value;
+            float correctionFactor = Toolbar.GetSetting<float>("CorrectionFactor").Value;
+            Enum.TryParse<BrightnessMode>((Toolbar.GetSetting<object>("Mode").Value as ComboBoxItem)?.Content as string, out var mode);
             Mode = mode;
 
             LayerChange[] layersChanges = new LayerChange[1];

PixiEditor/Models/Tools/Tools/CircleTool.cs

@@ -22,13 +22,13 @@ namespace PixiEditor.Models.Tools.Tools
 
         public override LayerChange[] Use(Layer layer, Coordinates[] coordinates, Color color)
         {
-            int thickness = (int) Toolbar.GetSetting("ToolSize").Value;
+            int thickness = Toolbar.GetSetting<int>("ToolSize").Value;
             DoubleCords fixedCoordinates = CalculateCoordinatesForShapeRotation(coordinates[^1], coordinates[0]);
             IEnumerable<Coordinates> outline = CreateEllipse(fixedCoordinates.Coords1, fixedCoordinates.Coords2, thickness);
             BitmapPixelChanges pixels = BitmapPixelChanges.FromSingleColoredArray(outline, color);
-            if ((bool) Toolbar.GetSetting("Fill").Value)
+            if (Toolbar.GetSetting<bool>("Fill").Value)
             {
-                Color fillColor = (Color) Toolbar.GetSetting("FillColor").Value;
+                Color fillColor = Toolbar.GetSetting<Color>("FillColor").Value;
                 pixels.ChangedPixels.AddRangeNewOnly(
                     BitmapPixelChanges.FromSingleColoredArray(CalculateFillForEllipse(outline), fillColor)
                         .ChangedPixels);

PixiEditor/Models/Tools/Tools/EraserTool.cs

@@ -2,7 +2,6 @@
 using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
-using PixiEditor.Models.Tools.ToolSettings;
 using PixiEditor.Models.Tools.ToolSettings.Toolbars;
 
 namespace PixiEditor.Models.Tools.Tools
@@ -19,7 +18,7 @@ namespace PixiEditor.Models.Tools.Tools
 
         public override LayerChange[] Use(Layer layer, Coordinates[] coordinates, Color color)
         {
-            return Erase(layer, coordinates, (int) Toolbar.GetSetting("ToolSize").Value);
+            return Erase(layer, coordinates, Toolbar.GetSetting<int>("ToolSize").Value);
         }
 
         public LayerChange[] Erase(Layer layer, Coordinates[] coordinates, int toolSize)

PixiEditor/Models/Tools/Tools/LineTool.cs

@@ -26,7 +26,7 @@ namespace PixiEditor.Models.Tools.Tools
             var pixels =
                 BitmapPixelChanges.FromSingleColoredArray(
                     CreateLine(coordinates, 
-                        (int) Toolbar.GetSetting("ToolSize").Value, CapType.Square, CapType.Square), color);
+                        Toolbar.GetSetting<int>("ToolSize").Value, CapType.Square, CapType.Square), color);
             return Only(pixels, layer);
         }
 

PixiEditor/Models/Tools/Tools/PenTool.cs

@@ -1,10 +1,9 @@
 using System.Windows.Input;
 using System.Windows.Media;
 using PixiEditor.Models.DataHolders;
-using PixiEditor.Models.Enums;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
-using PixiEditor.Models.Tools.ToolSettings;
+using PixiEditor.Models.Tools.ToolSettings.Settings;
 using PixiEditor.Models.Tools.ToolSettings.Toolbars;
 
 namespace PixiEditor.Models.Tools.Tools
@@ -12,20 +11,20 @@ namespace PixiEditor.Models.Tools.Tools
     public class PenTool : BitmapOperationTool
     {
         public override ToolType ToolType => ToolType.Pen;
-        private readonly int _toolSizeIndex;
+        private readonly Setting<int> _toolSizeSetting;
 
         public PenTool()
         {
             Cursor = Cursors.Pen;
             Tooltip = "Standard brush (B)";
             Toolbar = new BasicToolbar();
-            _toolSizeIndex = Toolbar.Settings.IndexOf(Toolbar.GetSetting("ToolSize"));
+            _toolSizeSetting = Toolbar.GetSetting<int>("ToolSize");
         }
 
         public override LayerChange[] Use(Layer layer, Coordinates[] coordinates, Color color)
         {
             Coordinates startingCords = coordinates.Length > 1 ? coordinates[1] : coordinates[0];
-            var pixels = Draw(startingCords, coordinates[0], color, (int) Toolbar.Settings[_toolSizeIndex].Value);
+            var pixels = Draw(startingCords, coordinates[0], color, _toolSizeSetting.Value);
             return Only(pixels, layer);
         }
 

PixiEditor/Models/Tools/Tools/RectangleTool.cs

@@ -21,12 +21,12 @@ namespace PixiEditor.Models.Tools.Tools
 
         public override LayerChange[] Use(Layer layer, Coordinates[] coordinates, Color color)
         {
-            int thickness = (int) Toolbar.GetSetting("ToolSize").Value;
+            int thickness = Toolbar.GetSetting<int>("ToolSize").Value;
             BitmapPixelChanges pixels =
                 BitmapPixelChanges.FromSingleColoredArray(CreateRectangle(coordinates, thickness), color);
-            if ((bool) Toolbar.GetSetting("Fill").Value)
+            if (Toolbar.GetSetting<bool>("Fill").Value)
             {
-                Color fillColor = (Color) Toolbar.GetSetting("FillColor").Value;
+                Color fillColor = Toolbar.GetSetting<Color>("FillColor").Value;
                 pixels.ChangedPixels.AddRangeOverride(
                     BitmapPixelChanges.FromSingleColoredArray
                             (CalculateFillForRectangle(coordinates[^1], coordinates[0], thickness), fillColor)

PixiEditor/Models/Tools/Tools/SelectTool.cs

@@ -6,7 +6,6 @@ using System.Windows.Input;
 using PixiEditor.Models.Controllers;
 using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.Enums;
-using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Tools.ToolSettings.Toolbars;
 using PixiEditor.ViewModels;
@@ -27,7 +26,7 @@ namespace PixiEditor.Models.Tools.Tools
 
         public override void OnMouseDown(MouseEventArgs e)
         {
-            Enum.TryParse((Toolbar.GetSetting("Mode").Value as ComboBoxItem)?.Content as string, out SelectionType);
+            Enum.TryParse((Toolbar.GetSetting<object>("Mode").Value as ComboBoxItem)?.Content as string, out SelectionType);
 
             _oldSelection = null;
             if (ViewModelMain.Current.ActiveSelection != null &&

PixiEditorTests/ModelsTests/ToolsTests/ToolbarTests/ToolbarBaseTests.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Text;
-using PixiEditor.Models.Tools.ToolSettings;
-using PixiEditor.Models.Tools.ToolSettings.Settings;
+using PixiEditor.Models.Tools.ToolSettings.Settings;
 using PixiEditor.Models.Tools.ToolSettings.Toolbars;
 using Xunit;
 
@@ -29,17 +25,17 @@ namespace PixiEditorTests.ModelsTests.ToolsTests.ToolbarTests
         {
             BasicToolbar toolbar = new BasicToolbar();
 
-            toolbar.Settings[0].Value = 5;
+            toolbar.GetSetting<int>("ToolSize").Value = 5;
 
             toolbar.SaveToolbarSettings();
 
             BasicShapeToolbar shapeToolbar = new BasicShapeToolbar();
 
-            Assert.NotEqual(5, (int)shapeToolbar.GetSetting("ToolSize").Value);
+            Assert.NotEqual(5, shapeToolbar.GetSetting<int>("ToolSize").Value);
 
             shapeToolbar.LoadSharedSettings();
 
-            Assert.Equal(5, (int)shapeToolbar.GetSetting("ToolSize").Value);
+            Assert.Equal(5, shapeToolbar.GetSetting<int>("ToolSize").Value);
         }
     }
 }