feat: add eraser tool

Author: RuntimeRascal

CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to GhostDraw will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## v1.0.6
+
+### Added
+- **Eraser Tool** - Remove drawing objects underneath the cursor
+  - Press `E` to activate Eraser tool
+  - Click and drag to erase drawings interactively
+  - Eraser size adjusts with mouse wheel (same as brush thickness)
+  - Intelligent intersection detection for precise erasing
+  - Works with both polylines (pen strokes) and lines (straight line tool)
+  - Custom eraser cursor with visual feedback
+- **Improved Code Quality**
+  - Enhanced tool interface consistency
+
+### Fixed
+- LineTool's `OnDeactivated` method now properly resets state without calling non-existent method
+
 ## v1.0.5
 
 ### Added
@@ -115,4 +131,3 @@ When adding new features or fixes:
 
 ### Fixed
 - Issue description and what was fixed
-```

Installer/GhostDraw.Installer.wixproj

@@ -1,6 +1,6 @@
 <Project Sdk="WixToolset.Sdk/4.0.5">
   <PropertyGroup>
-    <Version Condition="'$(Version)' == ''">1.0.5</Version>
+    <Version Condition="'$(Version)' == ''">1.0.6</Version>
     <OutputName>GhostDrawSetup-$(Version)</OutputName>
     <OutputType>Package</OutputType>
     <Platform>x64</Platform>

Src/GhostDraw/GhostDraw.csproj

@@ -8,7 +8,7 @@
     <UseWPF>true</UseWPF>
     <UseWindowsForms>true</UseWindowsForms>
     <ApplicationIcon>Assets\favicon.ico</ApplicationIcon>
-    <Version>1.0.5</Version>
+    <Version>1.0.6</Version>
   </PropertyGroup>
 
   <ItemGroup>

Src/GhostDraw/Tools/EraserTool.cs

@@ -1,30 +1,25 @@
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Input;
-using System.Windows.Media;
 using System.Windows.Shapes;
 using Microsoft.Extensions.Logging;
+using Point = System.Windows.Point;
 
 namespace GhostDraw.Tools;
 
 /// <summary>
 /// Eraser tool - removes drawing objects underneath the cursor
 /// </summary>
-public class EraserTool : IDrawingTool
+public class EraserTool(ILogger<EraserTool> logger) : IDrawingTool
 {
-    private readonly ILogger<EraserTool> _logger;
+    private readonly ILogger<EraserTool> _logger = logger;
     private bool _isErasing = false;
     private double _currentThickness = 3.0;
     private readonly HashSet<UIElement> _erasedElements = new();
-    
+
     // Tolerance for parallel line detection in intersection algorithm
     private const double PARALLEL_LINE_TOLERANCE = 0.001;
 
-    public EraserTool(ILogger<EraserTool> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnMouseDown(Point position, Canvas canvas)
     {
         _isErasing = true;

Src/GhostDraw/Tools/IDrawingTool.cs

@@ -1,6 +1,6 @@
-using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Input;
+using Point = System.Windows.Point;
 
 namespace GhostDraw.Tools;
 

Src/GhostDraw/Tools/LineTool.cs

@@ -1,29 +1,28 @@
-using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Input;
 using System.Windows.Media;
 using System.Windows.Shapes;
 using Microsoft.Extensions.Logging;
+using Point = System.Windows.Point;
+using Brush = System.Windows.Media.Brush;
+using Brushes = System.Windows.Media.Brushes;
+using Color = System.Windows.Media.Color;
+using ColorConverter = System.Windows.Media.ColorConverter;
 
 namespace GhostDraw.Tools;
 
 /// <summary>
 /// Straight line drawing tool - click two points to draw a line
 /// </summary>
-public class LineTool : IDrawingTool
+public class LineTool(ILogger<LineTool> logger) : IDrawingTool
 {
-    private readonly ILogger<LineTool> _logger;
+    private readonly ILogger<LineTool> _logger = logger;
     private Line? _currentLine;
     private Point? _lineStartPoint;
     private bool _isCreatingLine = false;
     private string _currentColor = "#FF0000";
     private double _currentThickness = 3.0;
 
-    public LineTool(ILogger<LineTool> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnMouseDown(Point position, Canvas canvas)
     {
         if (!_isCreatingLine)
@@ -61,42 +60,47 @@ public void OnActivated()
     public void OnDeactivated()
     {
         _logger.LogDebug("Line tool deactivated");
-        CancelCurrentLine();
+        if (_currentLine != null)
+        {
+            _currentLine = null;
+            _lineStartPoint = null;
+            _isCreatingLine = false;
+        }
     }
 
     public void OnColorChanged(string colorHex)
     {
         _currentColor = colorHex;
-        
+
         // Update in-progress line color if one exists
         if (_currentLine != null)
         {
             _currentLine.Stroke = CreateBrushFromHex(colorHex);
         }
-        
+
         _logger.LogDebug("Line color changed to {Color}", colorHex);
     }
 
     public void OnThicknessChanged(double thickness)
     {
         _currentThickness = thickness;
-        
+
         // Update in-progress line thickness if one exists
         if (_currentLine != null)
         {
             _currentLine.StrokeThickness = thickness;
         }
-        
+
         _logger.LogDebug("Line thickness changed to {Thickness}", thickness);
     }
 
     private void StartNewLine(Point startPoint, Canvas canvas)
     {
         _lineStartPoint = startPoint;
         _isCreatingLine = true;
-        
+
         var brush = CreateBrushFromHex(_currentColor);
-        
+
         _currentLine = new Line
         {
             X1 = startPoint.X,
@@ -108,7 +112,7 @@ private void StartNewLine(Point startPoint, Canvas canvas)
             StrokeStartLineCap = PenLineCap.Round,
             StrokeEndLineCap = PenLineCap.Round
         };
-        
+
         canvas.Children.Add(_currentLine);
         _logger.LogInformation("Line started at ({X:F0}, {Y:F0})", startPoint.X, startPoint.Y);
     }
@@ -119,17 +123,15 @@ private void FinishLine(Point endPoint)
         {
             _currentLine.X2 = endPoint.X;
             _currentLine.Y2 = endPoint.Y;
-            
+
             _logger.LogInformation("Line finished at ({X:F0}, {Y:F0})", endPoint.X, endPoint.Y);
         }
-        
+
         _currentLine = null;
         _lineStartPoint = null;
         _isCreatingLine = false;
     }
 
-
-
     private Brush CreateBrushFromHex(string colorHex)
     {
         try

Src/GhostDraw/Tools/PenTool.cs

@@ -1,27 +1,26 @@
-using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Input;
 using System.Windows.Media;
 using System.Windows.Shapes;
 using Microsoft.Extensions.Logging;
+using Point = System.Windows.Point;
+using Brush = System.Windows.Media.Brush;
+using Brushes = System.Windows.Media.Brushes;
+using Color = System.Windows.Media.Color;
+using ColorConverter = System.Windows.Media.ColorConverter;
 
 namespace GhostDraw.Tools;
 
 /// <summary>
 /// Freehand drawing tool
 /// </summary>
-public class PenTool : IDrawingTool
+public class PenTool(ILogger<PenTool> logger) : IDrawingTool
 {
-    private readonly ILogger<PenTool> _logger;
+    private readonly ILogger<PenTool> _logger = logger;
     private Polyline? _currentStroke;
     private string _currentColor = "#FF0000";
     private double _currentThickness = 3.0;
 
-    public PenTool(ILogger<PenTool> logger)
-    {
-        _logger = logger;
-    }
-
     public void OnMouseDown(Point position, Canvas canvas)
     {
         _logger.LogInformation("Starting new stroke at ({X:F0}, {Y:F0})", position.X, position.Y);

Src/GhostDraw/ViewModels/SettingsViewModel.cs

@@ -8,30 +8,23 @@ namespace GhostDraw.ViewModels;
 /// This enables proper MVVM pattern with dependency injection while keeping
 /// UserControls visible in the XAML designer.
 /// </summary>
-public class SettingsViewModel
+public class SettingsViewModel(
+    AppSettingsService appSettings,
+    LoggingSettingsService loggingSettings,
+    ILoggerFactory loggerFactory)
 {
     /// <summary>
     /// Service for managing application settings (brush, hotkey, mode, etc.)
     /// </summary>
-    public AppSettingsService AppSettings { get; }
-    
+    public AppSettingsService AppSettings { get; } = appSettings;
+
     /// <summary>
     /// Service for managing logging configuration
     /// </summary>
-    public LoggingSettingsService LoggingSettings { get; }
-    
+    public LoggingSettingsService LoggingSettings { get; } = loggingSettings;
+
     /// <summary>
     /// Factory for creating loggers for child controls
     /// </summary>
-    public ILoggerFactory LoggerFactory { get; }
-
-    public SettingsViewModel(
-        AppSettingsService appSettings,
-        LoggingSettingsService loggingSettings,
-        ILoggerFactory loggerFactory)
-    {
-        AppSettings = appSettings;
-        LoggingSettings = loggingSettings;
-        LoggerFactory = loggerFactory;
-    }
+    public ILoggerFactory LoggerFactory { get; } = loggerFactory;
 }

package.json

@@ -1,6 +1,6 @@
 {
     "name": "ghost-draw",
-    "version": "1.0.5",
+    "version": "1.0.6",
     "description": "Draw directly on your screen with a transparent overlay",
     "repository": {
         "type": "git",
@@ -24,7 +24,8 @@
         "format": "dotnet format GhostDraw.sln",
         "format:check": "dotnet format GhostDraw.sln --verify-no-changes",
         "version:check": "powershell.exe -NoProfile -ExecutionPolicy Bypass -File Scripts/Update-Version.ps1 -CreateTag -PushTag -UpdateProjFiles -Bump patch -DryRun",
-        "version:tag": "powershell.exe -NoProfile -ExecutionPolicy Bypass -File Scripts/Update-Version.ps1 -CreateTag -PushTag -Bump patch -DryRun"
+        "version:tag": "powershell.exe -NoProfile -ExecutionPolicy Bypass -File Scripts/Update-Version.ps1 -CreateTag -PushTag -Bump patch -DryRun",
+        "version:bump": "powershell.exe -NoProfile -ExecutionPolicy Bypass -File Scripts/Update-Version.ps1 -CreateTag -PushTag -UpdateProjFiles -Bump patch"
     },
     "engines": {
         "node": ">=18.0.0",