Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(video-player): enable autoplay & load as muted #12150

Merged
merged 4 commits into from
Dec 6, 2024

Conversation

m4olivei
Copy link
Contributor

@m4olivei m4olivei commented Dec 5, 2024

Related Ticket(s)

ADCMS-7129

Description

Ports a couple of v1 player fixes to v2 that got missed to fix auto-play while maintaining background-video mode:

Work that I've cherry-picked for this PR:
ADCMS-6363
#12070 (3d98787)
#12081 (863dfdc)

Changelog

New

  • Adds muted attribute to video-player-container
  • Adds storybook examples for auto-play and muted video player usage

Changed

  • Fixes auto-play attribute on video-player-container
  • Make background-video a reflected boolean attribute all the time.
  • Fix a regression with auto-play video.

@m4olivei m4olivei requested a review from a team as a code owner December 5, 2024 19:48
@m4olivei m4olivei changed the title Feat/video player muted feat(video-player): enable autoplay & load as muted Dec 5, 2024
Copy link

netlify bot commented Dec 5, 2024

Deploy Preview for ibm-dotcom-web-components ready!

Name Link
🔨 Latest commit a163e31
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components/deploys/67537d38571fe70008b239f3
😎 Deploy Preview https://deploy-preview-12150--ibm-dotcom-web-components.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 site configuration.

@andy-blum andy-blum added the Ready to merge Label for the pull requests that are ready to merge label Dec 6, 2024
@kodiakhq kodiakhq bot merged commit a68e9ba into release/v2.16.0 Dec 6, 2024
31 checks passed
@kodiakhq kodiakhq bot deleted the feat/video-player-muted branch December 6, 2024 23:00
Copy link
Contributor

Hey there! This issue/pull request was referenced in recently released v2.16.0.

1 similar comment
Copy link
Contributor

Hey there! This issue/pull request was referenced in recently released v2.16.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants