Skip to content

Standardize playback shortcuts - #44

Merged
austin-smith merged 1 commit into
mainfrom
standardize-playback-shortcuts
Aug 7, 2026
Merged

austin-smith merged 1 commit into
mainfrom
standardize-playback-shortcuts

Conversation

@austin-smith

Copy link
Copy Markdown
Owner

What changed

  • Keep bare left and right arrows focused on seeking by five seconds.
  • Add Ctrl+Left and Ctrl+Right as previous/next track aliases while retaining p and n.
  • Ignore held-key repeats for discrete track changes while allowing repeated seeking.
  • Document p/n as the compact canonical previous/next binding in the keymap overlay.
  • Add unit and headless terminal-input coverage for modifier handling and decoded arrow sequences.

Why

Bare arrows conventionally adjust position within the current media item, while a modifier distinguishes track navigation. Centralizing transport matching also prevents unsupported modifier combinations from being consumed accidentally.

User impact

Users can continue using p/n, use bare arrows to seek, or use Ctrl+arrows to change tracks. The persistent playback footer is unchanged.

Validation

  • bun run typecheck
  • bun test — 1,047 passed, 0 failed

@austin-smith
austin-smith marked this pull request as ready for review August 7, 2026 07:16
@austin-smith
austin-smith merged commit 668ad12 into main Aug 7, 2026
5 checks passed
@austin-smith
austin-smith deleted the standardize-playback-shortcuts branch August 7, 2026 07:27
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