refactor: select the previous tab while closing the actived one (#621)

love-linger · love-linger · commit 4a6cbddeac9d · 2024-10-30T09:53:58.000+08:00
Signed-off-by: leo &lt;longshuang@msn.cn&gt;
diff --git a/src/ViewModels/Launcher.cs b/src/ViewModels/Launcher.cs
@@ -204,12 +204,7 @@ public void CloseTab(LauncherPage page)
             var activeIdx = Pages.IndexOf(_activePage);
             if (removeIdx == activeIdx)
             {
-                ActivePage = Pages[removeIdx == Pages.Count - 1 ? removeIdx - 1 : removeIdx + 1];
-                CloseRepositoryInTab(page);
-                Pages.RemoveAt(removeIdx);
-            }
-            else if (removeIdx + 1 == activeIdx)
-            {
+                ActivePage = Pages[removeIdx > 0 ? removeIdx - 1 : removeIdx + 1];
                 CloseRepositoryInTab(page);
                 Pages.RemoveAt(removeIdx);
             }

Original file line number	Diff line number	Diff line change
`@@ -204,12 +204,7 @@ public void CloseTab(LauncherPage page)`
`204`	`204`	`var activeIdx = Pages.IndexOf(_activePage);`
`205`	`205`	`if (removeIdx == activeIdx)`
`206`	`206`	`{`
`207`		`- ActivePage = Pages[removeIdx == Pages.Count - 1 ? removeIdx - 1 : removeIdx + 1];`
`208`		`- CloseRepositoryInTab(page);`
`209`		`- Pages.RemoveAt(removeIdx);`
`210`		`- }`
`211`		`- else if (removeIdx + 1 == activeIdx)`
`212`		`- {`
	`207`	`+ ActivePage = Pages[removeIdx > 0 ? removeIdx - 1 : removeIdx + 1];`
`213`	`208`	`CloseRepositoryInTab(page);`
`214`	`209`	`Pages.RemoveAt(removeIdx);`
`215`	`210`	`}`