Skip to content

[codex] Add TV audio playback controls - #18

Draft
AndreKalberer wants to merge 1 commit into
mainfrom
codex/hbr005-tv-audio-controls
Draft

AndreKalberer wants to merge 1 commit into
mainfrom
codex/hbr005-tv-audio-controls

Conversation

@AndreKalberer

Copy link
Copy Markdown
Owner

Audio previews in the TV Listen player now expose Play/Pause, backward/forward 10-second controls, and elapsed/duration text. Readiness focuses Play/Pause; media keys support playback and bounded seeking, repeated keydown events are ignored, and unavailable seeking returns focus to Play/Pause. Errors expose retry, closing clears playback, and video routes keep their existing controls.

Pending play promises track both the source generation and individual attempt so deliberate pause, rapid resume, and source replacement do not turn stale AbortError rejections into an error screen. Genuine playback failures still expose retry. The new local-WAV regression runs as part of test:tv-smoke.

Validation:

  • npm run test:tv-smoke: TV navigation/catalog smoke and audio regression pass; the audio fixture covers readiness, remote media keys, repeat/keyup handling, seeking/replay, focus trapping, 1920/1280/720 layouts, delayed promise cancellation, genuine rejection, retry, Back cleanup, and movie separation.
  • Prior implementation verification passed source checks, synthetic configured-key audio checks, full integration review, and source/staged/extracted LG audio, HLS, and focus checks. Windows packaged checks also passed.
  • git diff --cached --check passes; exactly five source/test files are included.

The pre-existing native Android first-video and dual-key verification gap remains separate. Physical TV hardware was not tested. This draft does not include generated packages, versions, installers, or the separate staging fix.

This branch has not been deployed

No deployments
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