Fix: Save tab list on path change (#14874)

Author: yaira2

src/Files.App/Data/Contexts/Multitasking/MultitaskingContext.cs

@@ -45,18 +45,21 @@ private void AppModel_PropertyChanged(object? sender, PropertyChangedEventArgs e
 			if (e.PropertyName is nameof(AppModel.TabStripSelectedIndex))
 				UpdateCurrentTabIndex();
 		}
+
 		private void BaseMultitaskingControl_OnLoaded(object? sender, ITabBar control)
 		{
 			SetProperty(ref this.control, control, nameof(Control));
 			UpdateTabCount();
 			UpdateCurrentTabIndex();
 		}
+
 		private void HorizontalMultitaskingControl_SelectedTabItemChanged(object? sender, TabBarItem? e)
 		{
 			isPopupOpen = e is not null;
 			int newSelectedIndex = e is null ? currentTabIndex : MainPageViewModel.AppInstances.IndexOf(e);
 			UpdateSelectedTabIndex(newSelectedIndex);
 		}
+
 		private void FocusManager_GotFocus(object? sender, FocusManagerGotFocusEventArgs e)
 		{
 			if (isPopupOpen)
@@ -68,6 +71,7 @@ private void FocusManager_GotFocus(object? sender, FocusManagerGotFocusEventArgs
 				UpdateSelectedTabIndex(newSelectedIndex);
 			}
 		}
+
 		private void FocusManager_LosingFocus(object? sender, LosingFocusEventArgs e)
 		{
 			if (isPopupOpen)
@@ -83,13 +87,15 @@ private void UpdateTabCount()
 		{
 			SetProperty(ref tabCount, (ushort)MainPageViewModel.AppInstances.Count, nameof(TabCount));
 		}
+
 		private void UpdateCurrentTabIndex()
 		{
 			if (SetProperty(ref currentTabIndex, (ushort)App.AppModel.TabStripSelectedIndex, nameof(CurrentTabIndex)))
 			{
 				OnPropertyChanged(nameof(CurrentTabItem));
 			}
 		}
+
 		private void UpdateSelectedTabIndex(int index)
 		{
 			if (SetProperty(ref selectedTabIndex, (ushort)index, nameof(SelectedTabIndex)))

src/Files.App/Helpers/Navigation/NavigationHelpers.cs

@@ -61,9 +61,6 @@ public static async Task AddNewTabByPathAsync(Type type, string? path, int atInd
 			MainPageViewModel.AppInstances.Insert(index, tabItem);
 
 			App.AppModel.TabStripSelectedIndex = index;
-
-			// Save the updated tab list
-			AppLifecycleHelper.SaveSessionTabs();
 		}
 
 		public static async Task AddNewTabByParamAsync(Type type, object tabViewItemArgs, int atIndex = -1)
@@ -89,9 +86,6 @@ public static async Task AddNewTabByParamAsync(Type type, object tabViewItemArgs
 			var index = atIndex == -1 ? MainPageViewModel.AppInstances.Count : atIndex;
 			MainPageViewModel.AppInstances.Insert(index, tabItem);
 			App.AppModel.TabStripSelectedIndex = index;
-
-			// Save the updated tab list
-			AppLifecycleHelper.SaveSessionTabs();
 		}
 
 		private static async Task UpdateTabInfoAsync(TabBarItem tabItem, object navigationArg)

src/Files.App/Views/MainPage.xaml.cs

@@ -167,6 +167,9 @@ public async void TabItemContent_ContentChanged(object? sender, CustomTabViewIte
 			LoadPaneChanged();
 			UpdateNavToolbarProperties();
 			await NavigationHelpers.UpdateInstancePropertiesAsync(paneArgs);
+
+			// Save the updated tab list
+			AppLifecycleHelper.SaveSessionTabs();
 		}
 
 		public async void MultitaskingControl_CurrentInstanceChanged(object? sender, CurrentInstanceChangedEventArgs e)

src/Files.App/Views/PaneHolderPage.xaml.cs

@@ -166,9 +166,11 @@ public bool IsRightPaneVisible
 				{
 					_IsRightPaneVisible = value;
 					if (!_IsRightPaneVisible)
+					{
 						ActivePane = PaneLeft;
+						Pane_ContentChanged(null, null);
+					}
 
-					Pane_ContentChanged(null, null);
 					NotifyPropertyChanged(nameof(IsRightPaneVisible));
 					NotifyPropertyChanged(nameof(IsMultiPaneActive));
 				}