Skip to content

Commit ee36ed1

Browse files
refactor: Copilot review
[nitpick] The nested if-else structure could be simplified for better readability. Consider early return pattern: 'if (IsSecondaryWindow) { OnRequestClose?.Invoke(); return; }'
1 parent c2d339a commit ee36ed1

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/ViewModels/Launcher.cs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -284,9 +284,13 @@ public void CloseTab(LauncherPage page)
284284
else
285285
{
286286
if (IsSecondaryWindow)
287+
{
287288
OnRequestClose?.Invoke();
288-
else
289-
App.Quit(0);
289+
290+
return;
291+
}
292+
293+
App.Quit(0);
290294
}
291295

292296
return;

0 commit comments

Comments
 (0)