fix: Window resize now works on Linux

and the system has been uniformed on Windows/Linux to remove
the ShowWindowDecoration settings

Author: SebastienDuruz

src/WindowSwitcher.Lib/Data/Platform/Policies/LinuxSettingsPlatformPolicy.cs

@@ -11,7 +11,6 @@ public class ConfigFile
     public bool EnableSentry { get; set; } = true;
     public string SentryDsn { get; set; } = "https://d4530a120abfe1da5f25305d84502c0a@o4511093969584128.ingest.de.sentry.io/4511093982036048";
     public string TelemetryUserId { get; set; } = Guid.NewGuid().ToString("D");
-    public bool ShowWindowDecorations { get; set; } = false;
     public bool UseFixedWindowSize { get; set; } = false;
     public bool FocusOnHover { get; set; } = false;
     public int WindowWidth { get; set; } = 300;

src/WindowSwitcher.Lib/Data/Platform/Policies/WindowsSettingsPlatformPolicy.cs

@@ -5,13 +5,6 @@ namespace WindowSwitcher.Tests.Platform;
 
 public sealed class PlatformPoliciesTests
 {
-    [Fact]
-    public void LinuxAndWindowsSettingsPolicies_ExposeExpectedDecorationFlags()
-    {
-        Assert.True(new LinuxSettingsPlatformPolicy().ShowWindowDecorationSetting);
-        Assert.False(new WindowsSettingsPlatformPolicy().ShowWindowDecorationSetting);
-    }
-
     [Fact]
     public void LinuxAndWindowsPreviewPolicies_ExposeExpectedPreviewFlags()
     {

src/WindowSwitcher.Lib/Data/Platform/SystemInfo/Abstractions/ISettingsPlatformPolicy.cs

@@ -37,7 +37,6 @@ public static IServiceCollection AddPlatformServices(this IServiceCollection ser
                 IFloatingWindowHandleConfigurator,
                 WindowsFloatingWindowHandleConfigurator
             >();
-            services.AddSingleton<ISettingsPlatformPolicy, WindowsSettingsPlatformPolicy>();
             services.AddSingleton<IPlatformAppInfoProvider, WindowsPlatformAppInfoProvider>();
         }
         else if (OperatingSystem.IsLinux())
@@ -50,7 +49,6 @@ public static IServiceCollection AddPlatformServices(this IServiceCollection ser
                 IFloatingWindowHandleConfigurator,
                 NoOpFloatingWindowHandleConfigurator
             >();
-            services.AddSingleton<ISettingsPlatformPolicy, LinuxSettingsPlatformPolicy>();
             services.AddSingleton<IPlatformAppInfoProvider, LinuxPlatformAppInfoProvider>();
         }
         else

src/WindowSwitcher.Lib/Models/ConfigFile.cs

@@ -4,7 +4,6 @@
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
 using WindowSwitcher.Lib.Data;
-using WindowSwitcher.Lib.Data.Platform.SystemInfo.Abstractions;
 using WindowSwitcher.Lib.Models;
 using WindowSwitcher.Theming;
 
@@ -13,24 +12,19 @@ namespace WindowSwitcher.ViewModels;
 public class SettingsViewModel : ObservableObject
 {
     private readonly ConfigFileAccessor _configAccessor = ConfigFileAccessor.GetInstance();
-    private readonly ISettingsPlatformPolicy _settingsPlatformPolicy;
     private readonly Action _applyAction;
     private bool _pendingEnablePreviews;
     public IRelayCommand ApplyCommand { get; }
 
-    public SettingsViewModel(Action applyAction, ISettingsPlatformPolicy settingsPlatformPolicy)
+    public SettingsViewModel(Action applyAction)
     {
         ArgumentNullException.ThrowIfNull(applyAction);
-        ArgumentNullException.ThrowIfNull(settingsPlatformPolicy);
 
         _applyAction = applyAction;
-        _settingsPlatformPolicy = settingsPlatformPolicy;
         _pendingEnablePreviews = _configAccessor.ReadConfig(config => config.EnablePreviews);
         ApplyCommand = new RelayCommand(Apply);
     }
 
-    public bool ShowWindowDecorationSetting => _settingsPlatformPolicy.ShowWindowDecorationSetting;
-
     public bool EnablePreviews
     {
         get => _pendingEnablePreviews;
@@ -103,18 +97,6 @@ public bool FocusOnHover
             );
     }
 
-    public bool ShowWindowDecorations
-    {
-        get => ReadSetting(config => config.ShowWindowDecorations);
-        set =>
-            UpdateSetting(
-                nameof(ShowWindowDecorations),
-                value,
-                config => config.ShowWindowDecorations,
-                (config, currentValue) => config.ShowWindowDecorations = currentValue
-            );
-    }
-
     public bool UseFixedWindowSize
     {
         get => ReadSetting(config => config.UseFixedWindowSize);

src/WindowSwitcher.Tests/Platform/PlatformPoliciesTests.cs

@@ -19,7 +19,7 @@
         ExtendClientAreaChromeHints="NoChrome"
         ExtendClientAreaTitleBarHeightHint="-1"
         x:DataType="windows:MainWindow">
-        <Canvas Name="WindowCanvas" Background="{StaticResource FloatingWindowBackgroundBrush}" PointerPressed="CanvasPointerPressed" PointerReleased="CanvasPointerReleased" PointerEntered="CanvasPointerEntered" PointerExited="CanvasPointerExited">
+        <Canvas Name="WindowCanvas" Background="{StaticResource FloatingWindowBackgroundBrush}" PointerPressed="CanvasPointerPressed" PointerReleased="CanvasPointerReleased" PointerEntered="CanvasPointerEntered" PointerMoved="CanvasPointerMoved" PointerExited="CanvasPointerExited">
                 <Canvas.ContextMenu>
                         <ContextMenu Name="FloatingWindowContextMenu"></ContextMenu>
                 </Canvas.ContextMenu>

src/WindowSwitcher/Hosting/PlatformServiceCollectionExtensions.cs

@@ -18,6 +18,16 @@ namespace WindowSwitcher.Windows;
 
 public partial class FloatingWindow : Window, IFloatingPreviewWindow
 {
+    private const double ResizeGripThickness = 8d;
+    private static readonly Cursor DefaultCursor = new(StandardCursorType.Arrow);
+    private static readonly Cursor TopSideCursor = new(StandardCursorType.TopSide);
+    private static readonly Cursor BottomSideCursor = new(StandardCursorType.BottomSide);
+    private static readonly Cursor LeftSideCursor = new(StandardCursorType.LeftSide);
+    private static readonly Cursor RightSideCursor = new(StandardCursorType.RightSide);
+    private static readonly Cursor TopLeftCornerCursor = new(StandardCursorType.TopLeftCorner);
+    private static readonly Cursor TopRightCornerCursor = new(StandardCursorType.TopRightCorner);
+    private static readonly Cursor BottomLeftCornerCursor = new(StandardCursorType.BottomLeftCorner);
+    private static readonly Cursor BottomRightCornerCursor = new(StandardCursorType.BottomRightCorner);
     private volatile bool _isPointerInside;
     private bool _closeRequestedByHost;
     private readonly IFloatingWindowHost _floatingWindowHost;
@@ -119,10 +129,92 @@ private void SetInitialWindowSettings()
     private void CanvasPointerPressed(object? sender, PointerPressedEventArgs e)
     {
         _floatingWindowHost.SetActivePreview(this);
-        if (ConfigFileAccessor.GetInstance().ReadConfig(config => config.MoveWindows))
+
+        bool moveWindows = ConfigFileAccessor.GetInstance().ReadConfig(config => config.MoveWindows);
+
+        if (TryBeginResizeDrag(e))
+            return;
+
+        if (moveWindows)
             BeginMoveDrag(e);
     }
 
+    private bool TryBeginResizeDrag(PointerPressedEventArgs e)
+    {
+        if (!CanResize)
+            return false;
+
+        PointerPoint pointerPoint = e.GetCurrentPoint(this);
+        if (!pointerPoint.Properties.IsLeftButtonPressed)
+            return false;
+
+        WindowEdge? resizeEdge = ResolveResizeEdge(pointerPoint.Position);
+        if (!resizeEdge.HasValue)
+            return false;
+
+        BeginResizeDrag(resizeEdge.Value, e);
+        return true;
+    }
+
+    private WindowEdge? ResolveResizeEdge(Point pointerPosition)
+    {
+        if (Width <= 0 || Height <= 0)
+            return null;
+
+        bool isNearLeft = pointerPosition.X <= ResizeGripThickness;
+        bool isNearRight = pointerPosition.X >= Width - ResizeGripThickness;
+        bool isNearTop = pointerPosition.Y <= ResizeGripThickness;
+        bool isNearBottom = pointerPosition.Y >= Height - ResizeGripThickness;
+
+        if (isNearTop && isNearLeft)
+            return WindowEdge.NorthWest;
+        if (isNearTop && isNearRight)
+            return WindowEdge.NorthEast;
+        if (isNearBottom && isNearLeft)
+            return WindowEdge.SouthWest;
+        if (isNearBottom && isNearRight)
+            return WindowEdge.SouthEast;
+        if (isNearTop)
+            return WindowEdge.North;
+        if (isNearBottom)
+            return WindowEdge.South;
+        if (isNearLeft)
+            return WindowEdge.West;
+        if (isNearRight)
+            return WindowEdge.East;
+
+        return null;
+    }
+
+    private void UpdateResizeCursor(PointerEventArgs e)
+    {
+        if (!CanResize)
+        {
+            WindowCanvas.Cursor = DefaultCursor;
+            return;
+        }
+
+        PointerPoint pointerPoint = e.GetCurrentPoint(this);
+        WindowEdge? edge = ResolveResizeEdge(pointerPoint.Position);
+        WindowCanvas.Cursor = ResolveCursor(edge);
+    }
+
+    private static Cursor ResolveCursor(WindowEdge? edge)
+    {
+        return edge switch
+        {
+            WindowEdge.North => TopSideCursor,
+            WindowEdge.South => BottomSideCursor,
+            WindowEdge.West => LeftSideCursor,
+            WindowEdge.East => RightSideCursor,
+            WindowEdge.NorthWest => TopLeftCornerCursor,
+            WindowEdge.NorthEast => TopRightCornerCursor,
+            WindowEdge.SouthWest => BottomLeftCornerCursor,
+            WindowEdge.SouthEast => BottomRightCornerCursor,
+            _ => DefaultCursor,
+        };
+    }
+
     private void CanvasPointerReleased(object? sender, PointerReleasedEventArgs e)
     {
         _winAccessorBase.RaiseWindow(WindowConfig.WindowId);
@@ -131,6 +223,8 @@ private void CanvasPointerReleased(object? sender, PointerReleasedEventArgs e)
 
     private void CanvasPointerEntered(object? sender, PointerEventArgs e)
     {
+        UpdateResizeCursor(e);
+
         if (_isPointerInside)
             return;
         _isPointerInside = true;
@@ -154,6 +248,12 @@ private void CanvasPointerEntered(object? sender, PointerEventArgs e)
     private void CanvasPointerExited(object? sender, PointerEventArgs e)
     {
         _isPointerInside = false;
+        WindowCanvas.Cursor = DefaultCursor;
+    }
+
+    private void CanvasPointerMoved(object? sender, PointerEventArgs e)
+    {
+        UpdateResizeCursor(e);
     }
 
     private void FloatingWindowResized(object? sender, WindowResizedEventArgs e)
@@ -224,7 +324,6 @@ private void ApplySettingsCore(bool refreshPreviewPipeline)
                 config.UseFixedWindowSize,
                 config.WindowWidth,
                 config.WindowHeight,
-                config.ShowWindowDecorations,
                 config.PreviewHighlightColor,
             });
 
@@ -241,9 +340,10 @@ private void ApplySettingsCore(bool refreshPreviewPipeline)
             Height = WindowConfig.WindowHeight;
         }
 
-        SystemDecorations = configSnapshot.ShowWindowDecorations
-            ? SystemDecorations.Full
-            : SystemDecorations.BorderOnly;
+        if (!CanResize)
+            WindowCanvas.Cursor = DefaultCursor;
+
+        SystemDecorations = SystemDecorations.BorderOnly;
 
         if (Color.TryParse(configSnapshot.PreviewHighlightColor, out Color highlightColor))
         {

src/WindowSwitcher/ViewModels/SettingsViewModel.cs

@@ -22,9 +22,6 @@
                                   IsChecked="{Binding MoveWindows, Mode=TwoWay}" />
                         <CheckBox Content="Focus on hover"
                                   IsChecked="{Binding FocusOnHover, Mode=TwoWay}" />
-                        <CheckBox Content="Window decorations"
-                                  IsVisible="{Binding ShowWindowDecorationSetting}"
-                                  IsChecked="{Binding ShowWindowDecorations, Mode=TwoWay}" />
                         <CheckBox Content="Enable previews"
                                   IsChecked="{Binding EnablePreviews, Mode=TwoWay}" />
                         <CheckBox Content="Start minimized"