Skip to content

audio: Cmd/Alt+Shift+A cycles the live audio output#127

Merged
LinuxJedi merged 1 commit into
LinuxJedi:mainfrom
hobbo91:feature/audio-shortcut
Jul 6, 2026
Merged

audio: Cmd/Alt+Shift+A cycles the live audio output#127
LinuxJedi merged 1 commit into
LinuxJedi:mainfrom
hobbo91:feature/audio-shortcut

Conversation

@hobbo91

@hobbo91 hobbo91 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Adds a keyboard shortcut for the runtime "Audio Out" menu item (from #124): Cmd+Shift+A on macOS, Alt+Shift+A on Linux/Windows, cycling the live output forward -- Default -> host devices -> Disabled -- the same as the menu, reusing cycle_audio_output(). It's guarded so it only fires on a running machine, not the config-screen placeholder or while a menu/panel is open.

Shift (rather than a bare Cmd+A) matches the scheme's tier for infrequent, set-and-forget actions (save/load state, warp limit), and avoids the "Select All" muscle-memory collision accidentally silencing or misrouting audio.

Documented in the in-window Keyboard Shortcuts panel, the UI guide table, and the README audio section.

Verification: clippy -D warnings + fmt clean (default and --no-default-features); 1319 tests, 0 failures.

🤖 Generated with Claude Code

Add a keyboard shortcut for the runtime "Audio Out" menu item: Cmd+Shift+A
on macOS, Alt+Shift+A on Linux/Windows, cycling the live output forward
(Default -> host devices -> Disabled), the same as the menu and reusing
cycle_audio_output(). Ignored while a menu/panel is open, so it acts only on
a running machine, not the config-screen placeholder.

Shift matches the scheme's tier for infrequent, set-and-forget actions (save
state, load state, warp limit) and keeps the bare Cmd+A -- which collides with
"Select All" muscle memory -- from accidentally silencing or misrouting audio.

Documented in the Keyboard Shortcuts panel (panel grown one row), the UI guide
table, and the README audio section.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

@LinuxJedi LinuxJedi left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Good idea!

@LinuxJedi LinuxJedi merged commit 8def618 into LinuxJedi:main Jul 6, 2026
8 checks passed
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