performance: do not load diff content if CHANGES tab is not actived

love-linger · love-linger · commit 90a049d7412a · 2025-10-13T21:07:54.000+08:00
Signed-off-by: leo &lt;longshuang@msn.cn&gt;
diff --git a/src/ViewModels/CommitDetail.cs b/src/ViewModels/CommitDetail.cs
@@ -37,7 +37,12 @@ public int ActiveTabIndex
             set
             {
                 if (value != _sharedData.ActiveTabIndex)
+                {
                     _sharedData.ActiveTabIndex = value;
+
+                    if (value == 1 && DiffContext == null && _selectedChanges is { Count: 1 })
+                        DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption(_commit, _selectedChanges[0]));
+                }
             }
         }
 
@@ -94,7 +99,7 @@ public List<Models.Change> SelectedChanges
             {
                 if (SetProperty(ref _selectedChanges, value))
                 {
-                    if (value is not { Count: 1 })
+                    if (ActiveTabIndex != 1 || value is not { Count: 1 })
                         DiffContext = null;
                     else
                         DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption(_commit, value[0]), _diffContext);

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,12 @@ public int ActiveTabIndex`
`37`	`37`	`set`
`38`	`38`	`{`
`39`	`39`	`if (value != _sharedData.ActiveTabIndex)`
	`40`	`+ {`
`40`	`41`	`_sharedData.ActiveTabIndex = value;`
	`42`	`+`
	`43`	`+ if (value == 1 && DiffContext == null && _selectedChanges is { Count: 1 })`
	`44`	`+ DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption(_commit, _selectedChanges[0]));`
	`45`	`+ }`
`41`	`46`	`}`
`42`	`47`	`}`
`43`	`48`
`@@ -94,7 +99,7 @@ public List<Models.Change> SelectedChanges`
`94`	`99`	`{`
`95`	`100`	`if (SetProperty(ref _selectedChanges, value))`
`96`	`101`	`{`
`97`		`- if (value is not { Count: 1 })`
	`102`	`+ if (ActiveTabIndex != 1 \|\| value is not { Count: 1 })`
`98`	`103`	`DiffContext = null;`
`99`	`104`	`else`
`100`	`105`	`DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption(_commit, value[0]), _diffContext);`