Improve code quality

Author: Jack251970

Flow.Launcher/ViewModel/MainViewModel.cs

@@ -49,6 +49,8 @@ public partial class MainViewModel : BaseModel, ISavable
         private ChannelWriter<ResultsForUpdate> _resultsUpdateChannelWriter;
         private Task _resultsViewUpdateTask;
 
+        private readonly IReadOnlyList<Result> _emptyResult = new List<Result>();
+
         #endregion
 
         #region Constructor
@@ -830,16 +832,18 @@ private static string VerifyOrSetDefaultHotkey(string hotkey, string defaultHotk
 
         #region Preview
 
-        private bool? _selectedItemFromQueryResults;
+        private static readonly int ResultAreaColumnPreviewShown = 1;
+        private static readonly int ResultAreaColumnPreviewHidden = 3;
 
-        private ResultViewModel _selectedItem;
+        private bool? _selectedItemFromQueryResults;
 
+        private ResultViewModel _previewSelectedItem;
         public ResultViewModel PreviewSelectedItem
         {
-            get => _selectedItem;
+            get => _previewSelectedItem;
             set
             {
-                _selectedItem = value;
+                _previewSelectedItem = value;
                 OnPropertyChanged();
             }
         }
@@ -862,10 +866,6 @@ public bool InternalPreviewVisible
             }
         }
 
-        private static readonly int ResultAreaColumnPreviewShown = 1;
-
-        private static readonly int ResultAreaColumnPreviewHidden = 3;
-
         public int ResultAreaColumn { get; set; } = ResultAreaColumnPreviewShown;
 
         // This is not a reliable indicator of whether external preview is visible due to the
@@ -1016,7 +1016,7 @@ private bool QueryResultsPreviewed()
 
         #region Query
 
-        public void Query(bool isReQuery = false)
+        private void Query(bool isReQuery = false)
         {
             if (QueryResultsSelected())
             {
@@ -1121,8 +1121,6 @@ private void QueryHistory()
             }
         }
 
-        private readonly IReadOnlyList<Result> _emptyResult = new List<Result>();
-
         private async Task QueryResultsAsync(bool isReQuery = false, bool reSelect = true)
         {
             _updateSource?.Cancel();
@@ -1420,6 +1418,18 @@ public void ToggleFlowLauncher()
             }
         }
 
+        /// <summary>
+        /// Checks if Flow Launcher should ignore any hotkeys
+        /// </summary>
+        public bool ShouldIgnoreHotkeys()
+        {
+            return Settings.IgnoreHotkeysOnFullscreen && Win32Helper.IsForegroundWindowFullscreen() || GameModeStatus;
+        }
+
+        #endregion
+
+        #region Public Methods
+
         public void Show()
         {
             Application.Current.Dispatcher.Invoke(() =>
@@ -1505,7 +1515,7 @@ public async void Hide()
                 // 📌 Apply DWM Cloak (Completely hide the window)
                 Win32Helper.DWMSetCloakForWindow(mainWindow, true);
             }
-            
+
             await Task.Delay(50);
 
             // Update WPF properties
@@ -1518,17 +1528,8 @@ public async void Hide()
 #pragma warning restore VSTHRD100 // Avoid async void methods
 
         /// <summary>
-        /// Checks if Flow Launcher should ignore any hotkeys
+        /// Save history, user selected records and top most records
         /// </summary>
-        public bool ShouldIgnoreHotkeys()
-        {
-            return Settings.IgnoreHotkeysOnFullscreen && Win32Helper.IsForegroundWindowFullscreen() || GameModeStatus;
-        }
-
-        #endregion
-
-        #region Public Methods
-
         public void Save()
         {
             _historyItemsStorage.Save();