Skip to content

fix(core): use 0.2 and 0.7 for default playback rates#1236

Merged
mihar-22 merged 1 commit intomainfrom
t3code/ab76745e
Apr 7, 2026
Merged

fix(core): use 0.2 and 0.7 for default playback rates#1236
mihar-22 merged 1 commit intomainfrom
t3code/ab76745e

Conversation

@mihar-22
Copy link
Copy Markdown
Member

@mihar-22 mihar-22 commented Apr 7, 2026

Summary

  • Changes default sub-1x playback rates from 0.25/0.75 to 0.2/0.7
  • Updates tests and docs to match

Test plan

  • PlaybackRateButtonCore tests pass (18/18)

🤖 Generated with Claude Code


Note

Low Risk
Low risk: only changes the default playback rate list values (sub-1x rates) plus corresponding tests/docs, without altering cycling logic or media state handling.

Overview
Updates the default playback rate options to use 0.2x and 0.7x instead of 0.25x and 0.75x.

Aligns behavior expectations by updating playbackRateFeature defaults, PlaybackRateButtonCore tests (wrap/cycle assertions), and the PlaybackRateButton documentation to reflect the new rate list.

Reviewed by Cursor Bugbot for commit f9634a9. Bugbot is set up for automated code reviews on this repo. Configure here.

…ack rates

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v10-sandbox Ready Ready Preview, Comment Apr 7, 2026 5:18am

Request Review

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 7, 2026

Deploy Preview for vjs10-site ready!

Name Link
🔨 Latest commit f9634a9
🔍 Latest deploy log https://app.netlify.com/projects/vjs10-site/deploys/69d493a7a9341b0008352dda
😎 Deploy Preview https://deploy-preview-1236--vjs10-site.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mihar-22 mihar-22 marked this pull request as ready for review April 7, 2026 05:19
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

📦 Bundle Size Report

🎨 @videojs/html — no changes
Presets (7)
Entry Size
/video (default) 25.00 kB
/video (default + hls) 156.90 kB
/video (minimal) 24.97 kB
/video (minimal + hls) 156.88 kB
/audio (default) 23.16 kB
/audio (minimal) 23.15 kB
/background 6.93 kB
Media (7)
Entry Size
/media/background-video 1.04 kB
/media/container 1.59 kB
/media/dash-video 236.26 kB
/media/hls-video 133.40 kB
/media/mux-video 156.10 kB
/media/native-hls-video 3.54 kB
/media/simple-hls-video 15.06 kB
Players (3)
Entry Size
/video/player 6.65 kB
/audio/player 6.65 kB
/background/player 6.64 kB
Skins (17)
Entry Type Size
/video/minimal-skin.css css 3.42 kB
/video/skin.css css 3.44 kB
/video/minimal-skin js 24.99 kB
/video/minimal-skin.tailwind js 25.29 kB
/video/skin js 25.04 kB
/video/skin.tailwind js 25.40 kB
/audio/minimal-skin.css css 2.48 kB
/audio/skin.css css 2.45 kB
/audio/minimal-skin js 23.16 kB
/audio/minimal-skin.tailwind js 23.37 kB
/audio/skin js 23.13 kB
/audio/skin.tailwind js 23.45 kB
/background/skin.css css 117 B
/background/skin js 1.15 kB
/base.css css 157 B
/shared.css css 88 B
/skin-element js 1.34 kB
UI Components (22)
Entry Size
/ui/alert-dialog 2.22 kB
/ui/alert-dialog-close 1.73 kB
/ui/alert-dialog-description 1.62 kB
/ui/alert-dialog-title 1.63 kB
/ui/buffering-indicator 1.92 kB
/ui/captions-button 1.99 kB
/ui/controls 1.96 kB
/ui/error-dialog 2.44 kB
/ui/fullscreen-button 2.01 kB
/ui/mute-button 2.01 kB
/ui/pip-button 1.99 kB
/ui/play-button 2.00 kB
/ui/playback-rate-button 2.03 kB
/ui/popover 2.68 kB
/ui/poster 1.80 kB
/ui/seek-button 1.99 kB
/ui/slider 1.72 kB
/ui/thumbnail 2.25 kB
/ui/time 1.73 kB
/ui/time-slider 2.09 kB
/ui/tooltip 2.57 kB
/ui/volume-slider 2.42 kB

Sizes are marginal over the root entry point.

⚛️ @videojs/react — no changes
Presets (7)
Entry Size
/video (default) 19.98 kB
/video (default + hls) 152.16 kB
/video (minimal) 20.04 kB
/video (minimal + hls) 152.19 kB
/audio (default) 16.79 kB
/audio (minimal) 16.87 kB
/background 3.16 kB
Media (6)
Entry Size
/media/background-video 476 B
/media/dash-video 236.38 kB
/media/hls-video 133.35 kB
/media/mux-video 156.26 kB
/media/native-hls-video 3.50 kB
/media/simple-hls-video 15.03 kB
Skins (14)
Entry Type Size
/video/minimal-skin.css css 3.35 kB
/video/skin.css css 3.37 kB
/video/minimal-skin js 19.96 kB
/video/minimal-skin.tailwind js 23.41 kB
/video/skin js 19.93 kB
/video/skin.tailwind js 23.48 kB
/audio/minimal-skin.css css 2.38 kB
/audio/skin.css css 2.34 kB
/audio/minimal-skin js 16.79 kB
/audio/minimal-skin.tailwind js 19.26 kB
/audio/skin js 16.72 kB
/audio/skin.tailwind js 19.28 kB
/background/skin.css css 90 B
/background/skin js 272 B
UI Components (19)
Entry Size
/ui/alert-dialog 2.02 kB
/ui/buffering-indicator 1.75 kB
/ui/captions-button 1.87 kB
/ui/controls 1.72 kB
/ui/error-dialog 2.22 kB
/ui/fullscreen-button 1.88 kB
/ui/mute-button 1.88 kB
/ui/pip-button 1.88 kB
/ui/play-button 1.88 kB
/ui/playback-rate-button 1.91 kB
/ui/popover 3.47 kB
/ui/poster 1.60 kB
/ui/seek-button 1.90 kB
/ui/slider 3.71 kB
/ui/thumbnail 1.89 kB
/ui/time 1.92 kB
/ui/time-slider 3.23 kB
/ui/tooltip 2.78 kB
/ui/volume-slider 3.24 kB

Sizes are marginal over the root entry point.

🧩 @videojs/core — no changes
Entries (8)
Entry Size
. 5.29 kB
/dom 9.03 kB
/dom/media/custom-media-element 1.82 kB
/dom/media/dash 235.76 kB
/dom/media/hls 132.94 kB
/dom/media/mux 155.60 kB
/dom/media/native-hls 2.90 kB
/dom/media/simple-hls 14.45 kB
🏷️ @videojs/element — no changes
Entries (2)
Entry Size
. 999 B
/context 943 B
📦 @videojs/store — no changes
Entries (3)
Entry Size
. 1.39 kB
/html 696 B
/react 360 B
🔧 @videojs/utils — no changes
Entries (10)
Entry Size
/array 104 B
/dom 1.53 kB
/events 319 B
/function 327 B
/object 247 B
/predicate 265 B
/string 148 B
/style 190 B
/time 478 B
/number 158 B
📦 @videojs/spf — no changes
Entries (3)
Entry Size
. 40 B
/dom 12.45 kB
/playback-engine 12.41 kB

ℹ️ How to interpret

All sizes are standalone totals (minified + brotli).

Icon Meaning
No change
🔺 Increased ≤ 10%
🔴 Increased > 10%
🔽 Decreased
🆕 New (no baseline)

Run pnpm size locally to check current sizes.

@mihar-22 mihar-22 merged commit 932949f into main Apr 7, 2026
21 checks passed
@mihar-22 mihar-22 deleted the t3code/ab76745e branch April 7, 2026 05:25
@luwes luwes mentioned this pull request Apr 7, 2026
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