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

Use SettingsUtils.Default singleton for consistency

Browse Source 
Updated PowerDisplayPage, DashboardViewModel, and LightSwitchViewModel to use the SettingsUtils.Default singleton instance instead of creating new SettingsUtils objects. This change ensures consistent settings utility usage and improves resource management.
This commit is contained in:
Yu Leng
2025-12-09 14:21:29 +08:00
parent 9654ffde06
commit 10bdb31a8a
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/settings-ui/Settings.UI/SettingsXAML/Views/PowerDisplayPage.xaml.cs
View File

@@ -27,7 +27,7 @@ namespace Microsoft.PowerToys.Settings.UI.Views
public PowerDisplayPage()
{
var settingsUtils = new SettingsUtils();
var settingsUtils = SettingsUtils.Default;
ViewModel = new PowerDisplayViewModel(
settingsUtils,
SettingsRepository<GeneralSettings>.GetInstance(settingsUtils),

2
src/settings-ui/Settings.UI/ViewModels/DashboardViewModel.cs
View File

@@ -658,7 +658,7 @@ namespace Microsoft.PowerToys.Settings.UI.ViewModels
private ObservableCollection<DashboardModuleItem> GetModuleItemsPowerDisplay()
{
ISettingsRepository<PowerDisplaySettings> moduleSettingsRepository = SettingsRepository<PowerDisplaySettings>.GetInstance(new SettingsUtils());
ISettingsRepository<PowerDisplaySettings> moduleSettingsRepository = SettingsRepository<PowerDisplaySettings>.GetInstance(SettingsUtils.Default);
var settings = moduleSettingsRepository.SettingsConfig;
var list = new List<DashboardModuleItem>
{

2
src/settings-ui/Settings.UI/ViewModels/LightSwitchViewModel.cs
View File

@@ -723,7 +723,7 @@ namespace Microsoft.PowerToys.Settings.UI.ViewModels
{
try
{
var settingsUtils = new SettingsUtils();
var settingsUtils = SettingsUtils.Default;
var generalSettings = settingsUtils.GetSettingsOrDefault<GeneralSettings>(string.Empty);
IsPowerDisplayEnabled = generalSettings?.Enabled?.PowerDisplay ?? false;
Logger.LogInfo($"PowerDisplay enabled status: {IsPowerDisplayEnabled}");