Fix/tray icon flicker and block slider when HDR is on

[Cesarsk]

Hi @zpix1, I hope this is appreciated. I wanted to fix a couple of annoying issue.

PR Summary:

• Fix tray left-click behavior so startup and subsequent clicks use consistent open/close state.
• Keep the hidden startup webview parked offscreen without marking it as open.
• Move slow display/hardware commands off the Tauri UI thread.
• Keep monitor data visible during background refreshes and prevent overlapping reloads.
• Cache slow EDID/input capability probes.
• Avoid unnecessary brightness/input-source refetches on monitor switching.
• Reduce UI delays from debug serialization, slider animations, dialog close timers, and unnecessary control re-renders.
• Disable brightness controls when HDR is enabled.

Bug showed in a video (slow down the video to see it):

IMG_8236.MOV

[Copilot]

Pull request overview

This PR addresses UI/tray interaction issues in the Tauri app and prevents invalid brightness changes when HDR is enabled on a monitor.

Changes:

• Disable the brightness slider when hdr_status === "on" to block brightness adjustments under HDR.
• Rework tray left-click handling to toggle the main window open/closed and add a 300ms focus-out hide delay to reduce flicker/double-click issues.
• Dependency/lockfile updates (including package lock version bump and updated toolchain deps).

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/components/MonitorControls.tsx	Disables BrightnessSlider while HDR is enabled.
src-tauri/src/lib.rs	Implements tray left-click toggle logic and focus-out debounced hide.
src-tauri/src/winDisplays.rs	Minor formatting/import ordering and log formatting adjustments.
src-tauri/src/settings.rs	Formatting-only change (brace alignment).
src-tauri/capabilities/default.json	Formatting-only change (brace alignment).
package-lock.json	Updates lockfile versions/dependencies and app version field.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Cesarsk]

Hi @zpix1,

Addressed comments. New PR Summary:

• Fix tray left-click behavior so startup and subsequent clicks use consistent open/close state.
• Keep the hidden startup webview parked offscreen without marking it as open.
• Move slow display/hardware commands off the Tauri UI thread.
• Keep monitor data visible during background refreshes and prevent overlapping reloads.
• Cache slow EDID/input capability probes.
• Avoid unnecessary brightness/input-source refetches on monitor switching.
• Reduce UI delays from debug serialization, slider animations, dialog close timers, and unnecessary control re-renders.
• Disable brightness controls when HDR is enabled.

Feel free to review, thanks!

[zpix1]

what is it? is it required in lib.rs?

[zpix1]

Disable brightness controls when HDR is enabled.

Better add some description (title tooltip) to blocked brightness when HDR is on, something like "Brightness can't be adjusted when HDR is on"

Fix tray left-click behavior so startup and subsequent clicks use consistent open/close state.

Actually window hide on focus out is a feature, and it is not working now. You can extract option to settings (it actually exist as hidden one (shouldHideUIOnFocusOut))

Slider animations when switching monitors

It's a feature. If you don't like it, you can extract an option to settings, something like "reduced motion", and make it disabled by default.

Overall, great job!

[zpix1]

As stated above

[zpix1]

Also, please update changelog and bump version after fixes.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 15 changed files in this pull request and generated 1 comment.

[Cesarsk]

Hi @zpix1,

thanks for the detailed review.

I addressed the requested changes in commit fedc266:

• Added the HDR-disabled brightness tooltip with the text: Brightness can't be adjusted when HDR is on.
• Restored hide-on-focus-out behavior and exposed the existing shouldHideUIOnFocusOut setting in Settings as Hide controls when focus is lost.
• Kept the tray toggle fix intact by debouncing/cancelling focus-out hides when the tray toggle/reveal path is active.
• Restored slider animations. They still respect the OS prefers-reduced-motion setting. For now it's fine to not have it. I was fighting to understand how to speed up the UI on my Computer. I actually like the animation.
• Removed the WINDISPLAY_SMOKE block and smoke_report command from lib.rs; that was only local smoke-test instrumentation and is not required in the app -> Sorry about this, I haven't noticed it and I used it to do automatic tests on my side.

Thanks again for the review.