refactor: Improve commit selection logic and handling

- Refactor commit selection logic to track last selected commit separately and improve multi-selection handling.
- Refactor commit selection logic and remove commented auto-navigation code.
- Pass the selected commit item to the histories view model when the selection changes.

Author: heartacker

src/ViewModels/Histories.cs

@@ -24,11 +24,11 @@ public List<Models.Commit> Commits
             get => _commits;
             set
             {
-                var lastSelected = AutoSelectedCommit;
+                // LastSelectedCommit = AutoSelectedCommit;
                 if (SetProperty(ref _commits, value))
                 {
-                    if (value.Count > 0 && lastSelected != null)
-                        AutoSelectedCommit = value.Find(x => x.SHA == lastSelected.SHA);
+                    // if (value.Count > 0 && lastSelected != null)
+                    //     AutoSelectedCommit = value.Find(x => x.SHA == lastSelected.SHA);
                 }
             }
         }
@@ -45,6 +45,12 @@ public Models.Commit AutoSelectedCommit
             set => SetProperty(ref _autoSelectedCommit, value);
         }
 
+        public Models.Commit LastSelectedCommit
+        {
+            get => _lastSelectedCommit;
+            set => SetProperty(ref _lastSelectedCommit, value);
+        }
+
         public List<Models.Commit> LastSelectedCommits
         {
             get => _lastSelectedCommits;
@@ -104,6 +110,8 @@ public void Dispose()
             _repo = null;
             _graph = null;
             _autoSelectedCommit = null;
+            _lastSelectedCommits = null;
+            _lastSelectedCommit = null;
             _detailContext?.Dispose();
             _detailContext = null;
         }
@@ -158,19 +166,32 @@ public void NavigateTo(string commitSHA)
             });
         }
 
-        public void Select(IList commits)
+        /// <summary>
+        ///
+        /// notes: if multi selects, `AutoSelectedCommit` which `Binding Mode=OneWay` will not be updated,
+        /// so that we could not get the lastSelectedCommit automatically
+        ///
+        /// </summary>
+        /// <param name="commits"></param>
+        public void Select(IList commits, object selectedCommit)
         {
             if (commits.Count == 0)
             {
                 _repo.SelectedSearchedCommit = null;
                 DetailContext = null;
+                return;
             }
             else if (commits.Count == 1)
             {
                 var commit = (commits[0] as Models.Commit)!;
                 if (_repo.SelectedSearchedCommit == null || _repo.SelectedSearchedCommit.SHA != commit.SHA)
                     _repo.SelectedSearchedCommit = _repo.SearchedCommits.Find(x => x.SHA == commit.SHA);
 
+                var a = GetTopHeadsInCurHistory(commit, out var pathindex, 16);
+                // var ii = GetRelatedIndexsInCurHistory(_autoSelectedCommit);
+
+                a.Append(commit);
+                MarkLastSelectedCommitsAsSelected();
                 AutoSelectedCommit = commit;
                 NavigationId = _navigationId + 1;
 
@@ -199,12 +220,24 @@ public void Select(IList commits)
                 DetailContext = new Models.Count(commits.Count);
             }
 
+            LastSelectedCommit = selectedCommit as Models.Commit;
             _repo.SelectedCommits = commits.Cast<Models.Commit>().ToList();
         }
 
-        public void MarkCommitsAsSelected(IList<Models.Commit> commits)
+        public void MarkLastSelectedCommitsAsSelected(params IList<Models.Commit> commits)
         {
-            LastSelectedCommits = _commits.Where(x => commits.Any(y => y.SHA == x.SHA)).ToList();
+            var availableCommits = Commits.Where(x => commits.Any(y => y.SHA == x.SHA)).ToList();
+
+            // if LastSelectedCommits is not empty, find the AutoSelectedCommit or get last one
+            if (availableCommits.Count > 0 && LastSelectedCommit != null)
+            {
+                // find in oldAvailable or get last one
+                AutoSelectedCommit = availableCommits.Find(x => x.SHA == LastSelectedCommit.SHA)
+                                  ?? availableCommits.FirstOrDefault();
+            }
+            // set selectItem above(1item), now select others too
+            LastSelectedCommits = availableCommits;
+            NavigationId = _navigationId + 1;
         }
 
         public async Task<bool> CheckoutBranchByDecoratorAsync(Models.Decorator decorator)
@@ -417,6 +450,7 @@ private void NavigateTo(Models.Commit commit)
         private bool _isLoading = true;
         private List<Models.Commit> _commits = new List<Models.Commit>();
         private List<Models.Commit> _lastSelectedCommits = [];
+        private Models.Commit _lastSelectedCommit = null;
         private Models.CommitGraph _graph = null;
         private Models.Commit _autoSelectedCommit = null;
         private Models.Bisect _bisect = null;

src/ViewModels/Repository.cs

@@ -1379,10 +1379,12 @@ public void RefreshCommits()
 
                         BisectState = _histories.UpdateBisectInfo();
 
-                        _histories.MarkCommitsAsSelected(oldSelectedCommits);
+                        _histories.MarkLastSelectedCommitsAsSelected(oldSelectedCommits);
 
                         if (!string.IsNullOrEmpty(_navigateToCommitDelayed))
                             NavigateToCommit(_navigateToCommitDelayed);
+                        // else if (_histories.AutoSelectedCommit != null)
+                        //     NavigateToCommit(_histories.AutoSelectedCommit.SHA);
                     }
 
                     _navigateToCommitDelayed = string.Empty;

src/Views/Histories.axaml.cs

@@ -230,7 +230,7 @@ private void OnScrollToTopPointerPressed(object sender, PointerPressedEventArgs
         private void OnCommitListSelectionChanged(object _, SelectionChangedEventArgs e)
         {
             if (DataContext is ViewModels.Histories histories)
-                histories.Select(CommitListContainer.SelectedItems);
+                histories.Select(CommitListContainer.SelectedItems, CommitListContainer.SelectedItem);
 
             e.Handled = true;
         }