Improve code quality

Author: Jack251970

Flow.Launcher/SettingWindow.xaml.cs

@@ -1,6 +1,6 @@
 using System;
 using System.Windows;
-using System.Windows.Forms;
+using System.Windows.Controls;
 using System.Windows.Input;
 using System.Windows.Interop;
 using CommunityToolkit.Mvvm.DependencyInjection;
@@ -9,43 +9,53 @@
 using Flow.Launcher.SettingPages.Views;
 using Flow.Launcher.ViewModel;
 using ModernWpf.Controls;
-using TextBox = System.Windows.Controls.TextBox;
+using Screen = System.Windows.Forms.Screen;
 
 namespace Flow.Launcher;
 
 public partial class SettingWindow
 {
+    #region Private Fields
+
     private readonly Settings _settings;
-    private readonly SettingWindowViewModel _viewModel;
+
+    #endregion
+
+    #region Constructor
 
     public SettingWindow()
     {
-        var viewModel = Ioc.Default.GetRequiredService<SettingWindowViewModel>();
         _settings = Ioc.Default.GetRequiredService<Settings>();
+        var viewModel = Ioc.Default.GetRequiredService<SettingWindowViewModel>();
         DataContext = viewModel;
-        _viewModel = viewModel;
-        InitializePosition();
         InitializeComponent();
+
+        UpdatePositionAndState();
     }
 
+    #endregion
+
+    #region Window Events
+
     private void OnLoaded(object sender, RoutedEventArgs e)
     {
         RefreshMaximizeRestoreButton();
+
         // Fix (workaround) for the window freezes after lock screen (Win+L) or sleep
         // https://stackoverflow.com/questions/4951058/software-rendering-mode-wpf
         HwndSource hwndSource = PresentationSource.FromVisual(this) as HwndSource;
         HwndTarget hwndTarget = hwndSource.CompositionTarget;
         hwndTarget.RenderMode = RenderMode.SoftwareOnly;  // Must use software only render mode here
 
-        InitializePosition();
+        UpdatePositionAndState();
     }
 
     private void OnClosed(object sender, EventArgs e)
     {
         _settings.SettingWindowState = WindowState;
         _settings.SettingWindowTop = Top;
         _settings.SettingWindowLeft = Left;
-        _viewModel.Save();
+        _settings.Save();
         App.API.SavePluginSettings();
     }
 
@@ -104,7 +114,11 @@ private void Window_StateChanged(object sender, EventArgs e)
         RefreshMaximizeRestoreButton();
     }
 
-    public void InitializePosition()
+    #endregion
+
+    #region Window Position
+
+    public void UpdatePositionAndState()
     {
         var previousTop = _settings.SettingWindowTop;
         var previousLeft = _settings.SettingWindowLeft;
@@ -119,6 +133,7 @@ public void InitializePosition()
             Top = previousTop.Value;
             Left = previousLeft.Value;
         }
+
         WindowState = _settings.SettingWindowState;
     }
 
@@ -155,6 +170,10 @@ private double WindowTop()
         return top;
     }
 
+    #endregion
+
+    #region Navigation View Events
+
     private void NavigationView_SelectionChanged(NavigationView sender, NavigationViewSelectionChangedEventArgs args)
     {
         if (args.IsSettingsSelected)
@@ -201,4 +220,6 @@ private void ContentFrame_Loaded(object sender, RoutedEventArgs e)
     {
         NavView.SelectedItem ??= NavView.MenuItems[0]; /* Set First Page */
     }
+
+    #endregion
 }

Flow.Launcher/ViewModel/SettingWindowViewModel.cs

@@ -12,14 +12,6 @@ public SettingWindowViewModel(Settings settings)
         _settings = settings;
     }
 
-    /// <summary>
-    /// Save Flow settings. Plugins settings are not included.
-    /// </summary>
-    public void Save()
-    {
-        _settings.Save();
-    }
-
     public double SettingWindowWidth
     {
         get => _settings.SettingWindowWidth;

Flow.Launcher/WelcomeWindow.xaml.cs

@@ -2,16 +2,17 @@
 using System.Windows;
 using System.Windows.Input;
 using System.Windows.Controls;
-using Flow.Launcher.Resources.Pages;
-using ModernWpf.Media.Animation;
 using CommunityToolkit.Mvvm.DependencyInjection;
-using Flow.Launcher.ViewModel;
 using Flow.Launcher.Infrastructure.UserSettings;
+using Flow.Launcher.Resources.Pages;
+using Flow.Launcher.ViewModel;
+using ModernWpf.Media.Animation;
 
 namespace Flow.Launcher
 {
     public partial class WelcomeWindow : Window
     {
+        private readonly Settings _settings;
         private readonly WelcomeViewModel _viewModel;
 
         private readonly NavigationTransitionInfo _forwardTransitionInfo = new SlideNavigationTransitionInfo()
@@ -25,6 +26,7 @@ public partial class WelcomeWindow : Window
 
         public WelcomeWindow()
         {
+            _settings = Ioc.Default.GetRequiredService<Settings>();
             _viewModel = Ioc.Default.GetRequiredService<WelcomeViewModel>();
             DataContext = _viewModel;
             InitializeComponent();
@@ -97,7 +99,7 @@ private void ContentFrame_Loaded(object sender, RoutedEventArgs e)
         private void Window_Closed(object sender, EventArgs e)
         {
             // Save settings when window is closed
-            Ioc.Default.GetRequiredService<Settings>().Save();
+            _settings.Save();
         }
     }
 }