Skip to content

feat(player): add reel playback speed selector - #171

Merged
billyvg merged 2 commits into
masterfrom
feat/reel-playback-speed
Aug 24, 2026
Merged

feat(player): add reel playback speed selector#171
billyvg merged 2 commits into
masterfrom
feat/reel-playback-speed

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Adds a playback speed control to the screening reel player.

Why
Viewers can skim long reels faster without leaving the custom player controls.

What

  • Speed select in reel controls: 1x, 1.15x, 1.25x, 1.5x, 2x (default 1x)
  • Controller keeps the chosen rate on both dual-buffer video slots, including after attach/preload
  • Keyboard shortcuts ignore the speed <select> so space/arrow keys still work while focusing it

Checks

  • npm run test:app -- test/player/controller.test.tsx test/video-ui/video-ui.test.tsx
  • lint on touched player/test files

Requested by billy.

--

View Junior Session [Sentry]

Let screening reel viewers pick 1x–2x playback, and keep the rate on both dual-buffer slots across clip changes.

Co-Authored-By: Billy Vong <billy@sentry.io>
@billyvg
billyvg marked this pull request as ready for review August 24, 2026 13:53

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b2fd37f. Configure here.

Comment thread src/app/styles.css
Use background-color so the custom select caret is not cleared by the hover shorthand.
@billyvg
billyvg merged commit 484069f into master Aug 24, 2026
13 checks passed
@billyvg
billyvg deleted the feat/reel-playback-speed branch August 24, 2026 14:13
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