Mirrors/PowerToys
1
0
Fork 0
mirror of https://github.com/microsoft/PowerToys.git synced 2025-12-29 00:24:42 +01:00
Code Issues Packages Projects Releases Wiki Activity

Refactor SettingsDeepLink.OpenSettings signature

Browse Source 
Removed the unused SettingsWindow enum and refactored OpenSettings to accept only the mainExecutableIsOnTheParentFolder parameter. Updated all call sites to match the new method signature, simplifying settings navigation logic.
This commit is contained in:
Yu Leng
2025-12-26 16:30:49 +08:00
parent 92e26496f9
commit e88bae1bac
3 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/modules/powerdisplay/PowerDisplay/Helpers/SettingsDeepLink.cs
View File

@@ -9,12 +9,7 @@ namespace PowerDisplay.Helpers
{
public static class SettingsDeepLink
{
public enum SettingsWindow
{
PowerDisplay,
}
public static void OpenSettings(SettingsWindow window, bool mainExecutableIsOnTheParentFolder)
public static void OpenSettings(bool mainExecutableIsOnTheParentFolder)
{
try
{

2
src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs
View File

@@ -301,7 +301,7 @@ namespace PowerDisplay
{
// mainExecutableIsOnTheParentFolder = true because PowerDisplay is a WinUI 3 app
// deployed in a subfolder (PowerDisplay\) while PowerToys.exe is in the parent folder
SettingsDeepLink.OpenSettings(SettingsDeepLink.SettingsWindow.PowerDisplay, true);
SettingsDeepLink.OpenSettings(true);
}
/// <summary>

2
src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs
View File

@@ -299,7 +299,7 @@ namespace PowerDisplay
// Open PowerDisplay settings in PowerToys Settings UI
// mainExecutableIsOnTheParentFolder = true because PowerDisplay is a WinUI 3 app
// deployed in a subfolder (PowerDisplay\) while PowerToys.exe is in the parent folder
SettingsDeepLink.OpenSettings(SettingsDeepLink.SettingsWindow.PowerDisplay, true);
SettingsDeepLink.OpenSettings(true);
}
/// <summary>