Skip to content

feat(config): default preload to metadata - #1076

Merged
danieldanieli merged 2 commits into
masterfrom
feat/default-preload-metadata
Jul 30, 2026
Merged

feat(config): default preload to metadata#1076
danieldanieli merged 2 commits into
masterfrom
feat/default-preload-metadata

Conversation

@danieldanieli

Copy link
Copy Markdown
Contributor

Align the player default with standard browser behavior and cut the initial network cost: auto lets the browser pull media data for a video the user may never play, while metadata fetches only what is needed to report duration and dimensions.

Updated in both places the default is declared - defaults.js (runtime) and configSchema.json, which is copied into the published bundles as the documented option reference.

Note this governs the native

Align the player default with standard browser behavior and cut the
initial network cost: `auto` lets the browser pull media data for a video
the user may never play, while `metadata` fetches only what is needed to
report duration and dimensions.

Updated in both places the default is declared - defaults.js (runtime)
and configSchema.json, which is copied into the published bundles as the
documented option reference.

Note this governs the native <video> element, so it applies to
progressive sources; hls.js drives its own loading for adaptive
streaming, and autoplay still fetches media data either way.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@danieldanieli
danieldanieli requested a review from a team as a code owner July 29, 2026 09:56
@netlify

netlify Bot commented Jul 29, 2026

Copy link
Copy Markdown

Deploy Preview for cld-video-player ready!

Name Link
🔨 Latest commit 67597c9
🔍 Latest deploy log https://app.netlify.com/projects/cld-video-player/deploys/6a69e2f7f372100008936d00
😎 Deploy Preview https://deploy-preview-1076--cld-video-player.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.

@netlify

netlify Bot commented Jul 29, 2026

Copy link
Copy Markdown

Deploy Preview for cld-vp-esm-pages ready!

Name Link
🔨 Latest commit 67597c9
🔍 Latest deploy log https://app.netlify.com/projects/cld-vp-esm-pages/deploys/6a69e2f754ce380007ed1e4b
😎 Deploy Preview https://deploy-preview-1076--cld-vp-esm-pages.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.

@tsi tsi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@danieldanieli
danieldanieli merged commit 344f535 into master Jul 30, 2026
10 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