enhance: cleanup unused resources

Signed-off-by: leo <longshuang@msn.cn>

src/ViewModels/Launcher.cs

@@ -44,10 +44,10 @@ namespace SourceGit.ViewModels
             }
         }
 
-        public object Switcher
+        public IDisposable Switcher
         {
             get => _switcher;
-            set => SetProperty(ref _switcher, value);
+            private set => SetProperty(ref _switcher, value);
         }
 
         public Launcher(string startupRepo)
@@ -148,12 +148,13 @@ namespace SourceGit.ViewModels
 
         public void CancelSwitcher()
         {
+            Switcher?.Dispose();
             Switcher = null;
         }
 
         public void SwitchWorkspace(Workspace to)
         {
-            if (to.IsActive)
+            if (to == null || to.IsActive)
                 return;
 
             foreach (var one in Pages)
@@ -623,6 +624,6 @@ namespace SourceGit.ViewModels
         private LauncherPage _activePage = null;
         private bool _ignoreIndexChange = false;
         private string _title = string.Empty;
-        private object _switcher = null;
+        private IDisposable _switcher = null;
     }
 }

src/ViewModels/LauncherPageSwitcher.cs

@@ -4,7 +4,7 @@ using CommunityToolkit.Mvvm.ComponentModel;
 
 namespace SourceGit.ViewModels
 {
-    public class LauncherPageSwitcher : ObservableObject
+    public class LauncherPageSwitcher : ObservableObject, IDisposable
     {
         public List<LauncherPage> VisiblePages
         {
@@ -41,12 +41,17 @@ namespace SourceGit.ViewModels
 
         public void Switch()
         {
-            if (_selectedPage is { })
+            _launcher.ActivePage = _selectedPage ?? _launcher.ActivePage;
-                _launcher.ActivePage = _selectedPage;
-
             _launcher.CancelSwitcher();
         }
 
+        public void Dispose()
+        {
+            _visiblePages.Clear();
+            _selectedPage = null;
+            _searchFilter = string.Empty;
+        }
+
         private void UpdateVisiblePages()
         {
             var visible = new List<LauncherPage>();

src/ViewModels/WorkspaceSwitcher.cs

@@ -4,7 +4,7 @@ using CommunityToolkit.Mvvm.ComponentModel;
 
 namespace SourceGit.ViewModels
 {
-    public class WorkspaceSwitcher : ObservableObject
+    public class WorkspaceSwitcher : ObservableObject, IDisposable
     {
         public List<Workspace> VisibleWorkspaces
         {
@@ -41,12 +41,17 @@ namespace SourceGit.ViewModels
 
         public void Switch()
         {
-            if (_selectedWorkspace is { })
+            _launcher.SwitchWorkspace(_selectedWorkspace);
-                _launcher.SwitchWorkspace(_selectedWorkspace);
-
             _launcher.CancelSwitcher();
         }
 
+        public void Dispose()
+        {
+            _visibleWorkspaces.Clear();
+            _selectedWorkspace = null;
+            _searchFilter = string.Empty;
+        }
+
         private void UpdateVisibleWorkspaces()
         {
             var visible = new List<Workspace>();

src/Views/Launcher.axaml

@@ -104,9 +104,10 @@
     </ContentControl>
     
     <!-- Workspace/Pages Switcher -->
-    <Border Grid.Row="1"
+    <Border Grid.Row="0" Grid.RowSpan="2"
             Background="Transparent"
-            IsVisible="{Binding Switcher, Converter={x:Static ObjectConverters.IsNotNull}}">
+            IsVisible="{Binding Switcher, Converter={x:Static ObjectConverters.IsNotNull}}"
+            PointerPressed="OnCancelSwitcher">
       <Border HorizontalAlignment="Center" VerticalAlignment="Center" Effect="drop-shadow(0 0 12 #A0000000)">
         <Border Background="{DynamicResource Brush.Popup}" CornerRadius="8">
           <ContentControl Margin="16,10,16,12" Content="{Binding Switcher}">

src/Views/Launcher.axaml.cs

@@ -322,6 +322,13 @@ namespace SourceGit.Views
             e.Handled = true;
         }
 
+        private void OnCancelSwitcher(object sender, PointerPressedEventArgs e)
+        {
+            if (e.Source == sender)
+                (DataContext as ViewModels.Launcher)?.CancelSwitcher();
+            e.Handled = true;
+        }
+
         private KeyModifiers _unhandledModifiers = KeyModifiers.None;
         private WindowState _lastWindowState = WindowState.Normal;
     }

src/Views/LauncherTabBar.axaml

@@ -141,7 +141,7 @@
 
       <Button x:Name="PageSelector" Classes="icon_button" Width="16" Height="16" Margin="8,0">
         <Button.Flyout>
-          <Flyout Opened="OnTabsDropdownOpened">
+          <Flyout Opened="OnTabsDropdownOpened" Closed="OnTabsDropdownClosed">
             <Grid RowDefinitions="28,Auto" KeyDown="OnTabsDropdownKeyDown" LostFocus="OnTabsDropdownLostFocus">
               <TextBox Grid.Row="0"
                        Height="24"

src/Views/LauncherTabBar.axaml.cs

@@ -276,6 +276,12 @@ namespace SourceGit.Views
             UpdateSelectablePages();
         }
 
+        private void OnTabsDropdownClosed(object sender, EventArgs e)
+        {
+            SelectablePages.Clear();
+            SearchFilter = string.Empty;
+        }
+
         private void OnTabsDropdownKeyDown(object sender, KeyEventArgs e)
         {
             if (e.Key == Key.Escape)