Fixes #89: Bring back ordering of the currently selected column. Addi…

…tionally if no order is selected BuildOrder is used as defaultvalue.
StefanKert · Jul 17, 2020 · 1b9d837 · 1b9d837
1 parent 7b2f52a
commit 1b9d837
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/BuildVision.UI/ViewModels/BuildVisionPaneViewModel.cs b/src/BuildVision.UI/ViewModels/BuildVisionPaneViewModel.cs
@@ -209,7 +209,10 @@ public BuildVisionPaneViewModel(
             GroupedProjectsList.IsLiveGrouping = true;
             GroupedProjectsList.IsLiveSorting = true;
 
-            _buildInformationProvider.BuildStateChanged += () => { };
+            _buildInformationProvider.BuildStateChanged += () =>
+            {
+                GroupedProjectsList.Refresh();
+            };
 
             _settingsProvider = settingsProvider;
             _settingsProvider.SettingsChanged += () =>

diff --git a/src/BuildVision.UI/ViewModels/SortOrderFactory.cs b/src/BuildVision.UI/ViewModels/SortOrderFactory.cs
@@ -11,7 +11,7 @@ public class SortOrderFactory
         public static ProjectItemColumnSorter GetProjectItemSorter(SortDescription sortDescription)
         {
             var sortOrder = sortDescription.Order;
-            string sortPropertyName = sortDescription.Property;
+            var sortPropertyName = sortDescription.Property;
             if (sortOrder != SortOrder.None && !string.IsNullOrEmpty(sortPropertyName))
             {
                 ListSortDirection? sortDirection = sortOrder.ToSystem();
@@ -69,7 +69,7 @@ public static ProjectItemColumnSorter GetProjectItemSorter(SortDescription sortD
                         return new ProjectItemColumnSorter(sortDirection.Value, prop => prop.WarningsCount);
                 }
             }
-            return null;
+            return new ProjectItemColumnSorter(ListSortDirection.Ascending, prop => prop.BuildOrder);
         }
     }
 }