Skip to content

feat: improve tray quick connect feedback with in-app popups - #253

Closed
aaroncheungzy wants to merge 1 commit into
SpriteOvO:mainfrom
aaroncheungzy:codex/tray-battery-notifications
Closed

aaroncheungzy wants to merge 1 commit into
SpriteOvO:mainfrom
aaroncheungzy:codex/tray-battery-notifications

Conversation

@aaroncheungzy

Copy link
Copy Markdown
Contributor

Summary

With tray quick connect enabled, a single click now connects the selected device when disconnected and opens battery information when already connected. A successful connection automatically opens battery information, without an additional "Connected" notification.

  • replace Windows system notifications with a reusable application popup styled consistently with the battery window
  • retain in-app connection progress and error feedback; dismiss the popup when the battery window opens
  • handle single clicks immediately and remove the dedicated double-click action
  • update the setting description and all existing translation catalogs
  • add regression coverage for tray activation, connection outcomes, popup rendering, and the translated setting description

When quick connect is disabled, a single click continues to open the battery window. Right-click menu behavior is unchanged. No new dependencies or configuration keys are introduced.

Validation

  • Manually tested on Windows 10 with AirPods 4 with Active Noise Cancellation.
  • Local x64 RelWithDebInfo build passed with Qt 6.8.3 and MSVC 19.51 (Visual Studio 2026 Build Tools).
  • ctest --test-dir Build -C RelWithDebInfo --output-on-failure: 12/12 tests passed.
  • C++ formatting and git diff --check passed.

Screenshots

The following screenshots are captured by the local UI tests.

Light popup Dark popup
Light connection popup Dark connection popup

Updated Chinese setting description

@aaroncheungzy aaroncheungzy changed the title feat: show battery information after tray quick connect feat: improve tray quick connect feedback with in-app popups Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant