-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Request for comments: Shared video interface across AMP video players #3945
Comments
CCing list of AMP video player authors/maintainers @zeno @itaykinnrot @Em-PredragMilosevic @DarXector @esboyd @johnluther @mikepmtl @mister-ben @mkhatib @niallkennedy @pablos1 |
For more direct communication we'd like to create a video related discussion list (super low traffic). Please sign up here http://goo.gl/forms/fC09XfeLU5b2jXJr2
|
I filed #4113 and we will start working on it soon. I hope that it will both act as a proof of concept for this design and then help more implementations with example code. |
What's the latest status of this? is this a prioritized FR? Thanks |
@adelinamart We can close this. We already have the shared interface and it is implemented for youtube and amp-video. Other players would get their own GitHub issue. |
The AMP project is aiming to provide a better experience for consuming video in AMP documents. This includes features such as auto playing video, detaching videos from the scroll position and supporting analytics for video.
The planned features require a better awareness of AMP of the video playing status across all AMP elements that support video.
AMP supports a wide variety of video players, and we'd like to unify their treatment in AMP. If you are responsible for a video player please read the doc below and comment on whether:
Details are in this doc (and everybody should have comment access):
https://docs.google.com/document/d/1Lf2RmnUe_31lq7VNnG-GhIZWeOBijnrgJun67VTohmc/edit
CC @ampproject/video-on-amp @rudygalfi
The text was updated successfully, but these errors were encountered: