enhance: better undo/redo history in blame (#1686)

Signed-off-by: leo <[email protected]>

Author: love-linger

src/ViewModels/Blame.cs

@@ -76,7 +76,7 @@ public void Back()
             _navigationActiveIndex--;
             OnPropertyChanged(nameof(CanBack));
             OnPropertyChanged(nameof(CanForward));
-            NavigateToCommit(_navigationHistory[_navigationActiveIndex]);
+            NavigateToCommit(_navigationHistory[_navigationActiveIndex], true);
         }
 
         public void Forward()
@@ -87,21 +87,27 @@ public void Forward()
             _navigationActiveIndex++;
             OnPropertyChanged(nameof(CanBack));
             OnPropertyChanged(nameof(CanForward));
-            NavigateToCommit(_navigationHistory[_navigationActiveIndex]);
+            NavigateToCommit(_navigationHistory[_navigationActiveIndex], true);
         }
 
-        public void NavigateToCommit(string commitSHA)
+        public void NavigateToCommit(string commitSHA, bool isBackOrForward)
         {
-            if (!_navigationHistory[_navigationActiveIndex].Equals(commitSHA, StringComparison.Ordinal))
+            if (Revision.SHA.StartsWith(commitSHA, StringComparison.Ordinal))
+                return;
+
+            if (!isBackOrForward)
             {
+                var count = _navigationHistory.Count;
+                if (_navigationActiveIndex < count - 1)
+                    _navigationHistory.RemoveRange(_navigationActiveIndex + 1, count - _navigationActiveIndex - 1);
+
                 _navigationHistory.Add(commitSHA);
-                _navigationActiveIndex = _navigationHistory.Count - 1;
+                _navigationActiveIndex++;
                 OnPropertyChanged(nameof(CanBack));
                 OnPropertyChanged(nameof(CanForward));
             }
 
-            if (!Revision.SHA.StartsWith(commitSHA, StringComparison.Ordinal))
-                SetBlameData(commitSHA);
+            SetBlameData(commitSHA);
 
             if (App.GetLauncher() is { Pages: { } pages })
             {

src/Views/Blame.axaml.cs

@@ -223,7 +223,7 @@ protected override void OnPointerPressed(PointerPressedEventArgs e)
                         if (rect.Contains(pos))
                         {
                             if (DataContext is ViewModels.Blame blame)
-                                blame.NavigateToCommit(info.CommitSHA);
+                                blame.NavigateToCommit(info.CommitSHA, false);
 
                             e.Handled = true;
                             break;