From db1ecf4a516ac0b539efcf2ecf71159a895876e0 Mon Sep 17 00:00:00 2001
From: Andreas Weizel <rpinski@awzhome.de>
Date: Wed, 7 Aug 2013 00:49:28 +0200
Subject: [PATCH] When list of AddIns (installed or in repository) gets
 updated, view is trying to restore the previous item selection.

---
 .../Project/Src/View/AddInsView.xaml          |  8 +++++++-
 .../ViewModel/AddInPackageViewModelBase.cs    |  6 ++++++
 .../Src/ViewModel/AddInsViewModelBase.cs      | 20 +++++++++++++++++++
 .../Src/ViewModel/NuGetAddInsViewModelBase.cs |  2 +-
 4 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
index 9acd6404d9..5ba8aafd61 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/View/AddInsView.xaml
@@ -119,6 +119,10 @@
 			</Style.Triggers>-->
 		</Style>
 		
+		<Style x:Key="packageListBoxItemContainerStyle" TargetType="{x:Type ListBoxItem}">
+			<Setter Property="IsSelected" Value="{Binding Mode=TwoWay, Path=IsSelected}"/>
+		</Style>
+		
 		<DataTemplate x:Key="PackageListBoxItemTemplate">
 			<Grid>
 				<Grid.ColumnDefinitions>
@@ -491,7 +495,7 @@
 						<RowDefinition Height="Auto"/>
 					</Grid.RowDefinitions>
 					
-<!--				<TextBlock
+					<!--				<TextBlock
 						Grid.Row="0"
 						Grid.Column="0"
 						VerticalAlignment="Center"
@@ -550,8 +554,10 @@
 						x:Name="allPackagesListBox"
 						Grid.Row="0"
 						SelectedIndex="0"
+						IsSynchronizedWithCurrentItem="true"
 						ItemTemplate="{StaticResource PackageListBoxItemTemplate}"
 						ItemsSource="{Binding Path=AddInPackages}"
+						ItemContainerStyle="{StaticResource packageListBoxItemContainerStyle}"
 						HorizontalContentAlignment="Stretch"
 						ScrollViewer.HorizontalScrollBarVisibility="Disabled"/>
 					<ProgressBar
diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInPackageViewModelBase.cs b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInPackageViewModelBase.cs
index 616283c131..a90dbe6d19 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInPackageViewModelBase.cs
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInPackageViewModelBase.cs
@@ -252,6 +252,12 @@ namespace ICSharpCode.AddInManager2.ViewModel
 				return false;
 			}
 		}
+		
+		public bool IsSelected
+		{
+			get;
+			set;
+		}
 
 		public virtual void AddPackage()
 		{
diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInsViewModelBase.cs b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInsViewModelBase.cs
index 3156dd7d24..3619b09abc 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInsViewModelBase.cs
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/AddInsViewModelBase.cs
@@ -23,6 +23,7 @@ namespace ICSharpCode.AddInManager2.ViewModel
 		private bool _isReadingPackages;
 		private bool _isExpandedinView;
 		private bool _showPackageSources;
+		private string _lastSelectedId;
 		
 		private ObservableCollection<PackageRepository> _packageRepositories;
 		
@@ -289,9 +290,11 @@ namespace ICSharpCode.AddInManager2.ViewModel
 		
 		protected void UpdatePackageViewModels(IEnumerable<AddInPackageViewModelBase> newPackageViewModels)
 		{
+			StoreSelection();
 			ClearPackages();
 			NuGet.CollectionExtensions.AddRange(AddInPackages, newPackageViewModels);
 			UpdateInstallationState();
+			RestoreSelection();
 		}
 		
 		protected void ClearPackages()
@@ -545,5 +548,22 @@ namespace ICSharpCode.AddInManager2.ViewModel
 			// Update the list of package sources
 			UpdatePackageSources();
 		}
+		
+		private void StoreSelection()
+		{
+			AddInPackageViewModelBase selectedModel = AddInPackages.FirstOrDefault(m => m.IsSelected);
+			if (selectedModel != null) {
+				_lastSelectedId = selectedModel.Id;
+			}
+		}
+		
+		private void RestoreSelection()
+		{
+			AddInPackageViewModelBase modelToSelect = AddInPackages.FirstOrDefault(m => m.Id == _lastSelectedId);
+			if (modelToSelect != null) {
+				modelToSelect.IsSelected = true;
+			}
+			_lastSelectedId = null;
+		}
 	}
 }
diff --git a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/NuGetAddInsViewModelBase.cs b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/NuGetAddInsViewModelBase.cs
index 2a549a9932..ebe779244c 100644
--- a/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/NuGetAddInsViewModelBase.cs
+++ b/src/AddIns/Misc/AddInManager2/Project/Src/ViewModel/NuGetAddInsViewModelBase.cs
@@ -49,7 +49,7 @@ namespace ICSharpCode.AddInManager2.ViewModel
 		{
 			IsReadingPackages = true;
 			HasError = false;
-			ClearPackages();
+//			ClearPackages();
 			CancelReadPackagesTask();
 			CreateReadPackagesTask();
 			_task.Start();