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

Request for comments: Shared video interface across AMP video players #3945

Closed
cramforce opened this issue Jul 7, 2016 · 5 comments
Closed
Assignees
Labels
INTENT TO IMPLEMENT Proposes implementation of a significant new feature. https://bit.ly/amp-contribute-code Type: Discussion/Question

Comments

@cramforce
Copy link
Member

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:

  • this makes sense from your point of view.
  • this can be implemented with your video player.
  • we are missing something that is important with respect to video in your experience.
  • anything else that might be relevant.

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

@cramforce cramforce added Type: Discussion/Question INTENT TO IMPLEMENT Proposes implementation of a significant new feature. https://bit.ly/amp-contribute-code Related to: Content labels Jul 7, 2016
@cramforce
Copy link
Member Author

CCing list of AMP video player authors/maintainers @zeno @itaykinnrot @Em-PredragMilosevic @DarXector @esboyd @johnluther @mikepmtl @mister-ben @mkhatib @niallkennedy @pablos1

@cramforce
Copy link
Member Author

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

Mentions

@zeno @itaykinnrot @Em-PredragMilosevic @DarXector @esboyd @johnluther @mikepmtl @mister-ben @mkhatib @niallkennedy @pablos1

@cramforce
Copy link
Member Author

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.

@adelinamart
Copy link
Contributor

What's the latest status of this? is this a prioritized FR? Thanks

@aghassemi
Copy link
Contributor

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTENT TO IMPLEMENT Proposes implementation of a significant new feature. https://bit.ly/amp-contribute-code Type: Discussion/Question
Projects
None yet
Development

No branches or pull requests

4 participants