Skip to content

play user action required sound regardless of window focus#292930

Merged
meganrogge merged 1 commit intomainfrom
stable-salamander
Feb 4, 2026
Merged

play user action required sound regardless of window focus#292930
meganrogge merged 1 commit intomainfrom
stable-salamander

Conversation

@meganrogge
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 4, 2026 20:36
@meganrogge meganrogge self-assigned this Feb 4, 2026
@meganrogge meganrogge added this to the January 2026 milestone Feb 4, 2026
@meganrogge meganrogge enabled auto-merge (squash) February 4, 2026 20:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts chat “user action required” signaling so the accessibility signal is played even when the VS Code window is focused, while keeping OS toast notifications gated to the unfocused-window case.

Changes:

  • Move AccessibilitySignal.chatUserActionRequired playback to occur before the window-focus early return.
  • Preserve existing behavior where OS toast + window attention only happens when the target window is not focused.

@meganrogge meganrogge merged commit 48c2726 into main Feb 4, 2026
27 of 28 checks passed
@meganrogge meganrogge deleted the stable-salamander branch February 4, 2026 20:58
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.

2 participants