fix(docs): enable muted autoplay for YouTube video in “Watch & Learn#641
Conversation
|
@error9098x is attempting to deploy a commit to the Cloudinary DevX Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi @error9098x ! Thank you for this contribution. I don't think we actually want the video to autoplay. Thank you for catching that we were using the autoPlay attribute, and making it actually functional -- however I have edited your PR to strip the attribute entirely instead, and am merging it. Thanks! |
6f1c5e4
into
cloudinary-community:main
|
@all-contributors please add @error9098x for docs |
|
This project's configuration file has malformed JSON: .all-contributorsrc. Error:: Unexpected string in JSON at position 11236 |
|
🎉 This PR is included in version 6.17.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
@
No problem, makes sense. Thanks for using my PR for fixing the issue :) |
Description
Enable muted autoplay for the YouTube video in the docs “Watch & Learn” section to improve UX. Autoplay is requested via YouTube URL params (required by modern browser policies), and the redundant
autoPlayattribute is removed.next-cloudinary/docs/components/Video/Video.js/cldimage/basic-usage→ “Watch & Learn”Before vs After
Code change
No new dependencies.
Issue Ticket Number
Fixes #640
Type of change
Checklist