Made both hardware event types togglable in debug settings

xanderfrangos · Jan 18, 2024 · 729a2b0 · 729a2b0
1 parent 91f4e13
commit 729a2b0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/components/SettingsWindow.jsx b/src/components/SettingsWindow.jsx
@@ -1374,6 +1374,8 @@ export default class SettingsWindow extends PureComponent {
                                     <SettingsOption title="VCP read delay" description="How long (in miliseconds) to delay returning a VCP code value. This can help some displays not return random errors." input={<input type="number" min="0" max="200" value={this.state.rawSettings.checkVCPWaitMS * 1} onChange={(e) => this.setSetting("checkVCPWaitMS", e.target.value)} /> } />
 
                                     <SettingsOption title="Disable Auto Refresh" description="Prevent last known brightness from read after certain hardware/user events." input={this.renderToggle("disableAutoRefresh")} />
+                                    <SettingsOption title="Use Win32 hardware events" input={this.renderToggle("useWin32Event")} />
+                                    <SettingsOption title="Use Electron hardware events" input={this.renderToggle("useElectronEvents")} />
                                     <SettingsOption title="Show console window (requires restart)" input={this.renderToggle("showConsole")} />
                                     <SettingsOption title="Use Taskbar Registry" input={this.renderToggle("useTaskbarRegistry")} />
                                     <SettingsOption title="Disable Mouse Events (requires restart)" input={this.renderToggle("disableMouseEvents")} />

diff --git a/src/electron.js b/src/electron.js
@@ -449,6 +449,8 @@ const defaultSettings = {
   disableWMI: false,
   disableWin32: false,
   autoDisabledWMI: false,
+  useWin32Event: true,
+  useElectronEvents: true,
   defaultOverlayType: "safe",
   disableMouseEvents: false,
   disableThrottling: false,
@@ -3469,10 +3471,10 @@ function addEventListeners() {
   systemPreferences.on('color-changed', handleAccentChange)
   nativeTheme.on('updated', handleAccentChange)
 
-  addDisplayChangeListener(() => handleMonitorChange("win32"))
-  screen.addListener("display-added", () => handleMonitorChange("display-added"))
-  screen.addListener("display-removed", () => handleMonitorChange("display-removed"))
-  screen.addListener("display-metrics-changed", () => handleMonitorChange("display-metrics-changed"))
+  addDisplayChangeListener(() => { if(settings.useWin32Event) handleMonitorChange("win32") })
+  screen.addListener("display-added", () => { if(settings.useElectronEvents) handleMonitorChange("display-added") })
+  screen.addListener("display-removed", () => { if(settings.useElectronEvents) handleMonitorChange("display-removed") })
+  screen.addListener("display-metrics-changed", () => { if(settings.useElectronEvents) handleMonitorChange("display-metrics-changed") })
 
   enableMouseEvents()