Added safeguards for edge cases

goran-w · goran-w · commit 5283a06acd6d · 2024-11-29T09:04:41.000+01:00
diff --git a/src/Views/TextDiffView.axaml.cs b/src/Views/TextDiffView.axaml.cs
@@ -1145,6 +1145,8 @@ public class SingleSideTextDiffPresenter : ThemedTextDiffPresenter
 
         public void ForceSyncScrollOffset()
         {
+            if (_scrollViewer == null)
+                return;
             if (DataContext is ViewModels.TwoSideTextDiff diff)
                 diff.SyncScrollOffset = _scrollViewer?.Offset ?? Vector.Zero;
         }
@@ -1584,7 +1586,7 @@ static TextDiffView()
 
             CurrentChangeBlockIdxProperty.Changed.AddClassHandler<TextDiffView>((v, e) =>
             {
-                if (v.Editor.Presenter != null)
+                if ((int)e.NewValue >= 0 && v.Editor.Presenter != null)
                 {
                     foreach (var p in v.Editor.Presenter.GetVisualDescendants().OfType<ThemedTextDiffPresenter>())
                     {

Original file line number	Diff line number	Diff line change
`@@ -1145,6 +1145,8 @@ public class SingleSideTextDiffPresenter : ThemedTextDiffPresenter`
`1145`	`1145`
`1146`	`1146`	`public void ForceSyncScrollOffset()`
`1147`	`1147`	`{`
	`1148`	`+ if (_scrollViewer == null)`
	`1149`	`+ return;`
`1148`	`1150`	`if (DataContext is ViewModels.TwoSideTextDiff diff)`
`1149`	`1151`	`diff.SyncScrollOffset = _scrollViewer?.Offset ?? Vector.Zero;`
`1150`	`1152`	`}`
`@@ -1584,7 +1586,7 @@ static TextDiffView()`
`1584`	`1586`
`1585`	`1587`	`CurrentChangeBlockIdxProperty.Changed.AddClassHandler<TextDiffView>((v, e) =>`
`1586`	`1588`	`{`
`1587`		`- if (v.Editor.Presenter != null)`
	`1589`	`+ if ((int)e.NewValue >= 0 && v.Editor.Presenter != null)`
`1588`	`1590`	`{`
`1589`	`1591`	`foreach (var p in v.Editor.Presenter.GetVisualDescendants().OfType<ThemedTextDiffPresenter>())`
`1590`	`1592`	`{`