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

Jump to bottom

Support video-interface and autoplay for amp-vimeo. #11319

Closed
aghassemi opened this issue Sep 18, 2017 · 4 comments
Closed

Support video-interface and autoplay for amp-vimeo. #11319

aghassemi opened this issue Sep 18, 2017 · 4 comments
Assignees
@alanorozco
Labels
Blocked Type: Feature Request
Projects
UI
Milestone
New FRs

Comments

@aghassemi
Copy link
Contributor

aghassemi commented Sep 18, 2017 •
edited

Should be fairly similar to <amp-dailymotion> if Vimeo has an API into this.
@aghassemi aghassemi added this to the New FRs milestone Sep 18, 2017
@wassgha
Copy link
Contributor

wassgha commented Oct 6, 2017 •
edited

From my research on the Vimeo API, "playing" the video will automatically make it enter fullscreen on mobile (simulates the old iOS behavior). This prevents autoplay and video docking.
See API demo here

@aghassemi
Copy link
Contributor Author

yeah : ( https://vimeo.com/forums/topic:268198 https://vimeo.com/forums/feature_requests/topic:100233

For a consistent user experience with the Vimeo Player, we do not allow autoplay or inline playback on mobile, and for the immediate future will always require the user to initiate playback. Apologies for the trouble!

I don't get it.

@ericlindley-g ericlindley-g added this to Needs Triage in UI Jan 5, 2018
@ericlindley-g ericlindley-g moved this from Needs Triage to 3P integrations in UI Jan 5, 2018
@alanorozco
Copy link
Member

alanorozco commented Apr 12, 2018 •
edited

It looks like some things have changed now:

  • No longer goes into fullscreen on play.
  • Autoplay is allowed, but "Please be aware that autoplay and loop will not work on most mobile devices." <- which I think is already the case for <amp-video>?

@alanorozco alanorozco self-assigned this Apr 13, 2018
@aghassemi
Copy link
Contributor Author

#15326 good enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alanorozco alanorozco

Labels
Blocked Type: Feature Request
Projects
No open projects
UI
  
3P integrations
Milestone
New FRs
Development

No branches or pull requests
3 participants
@alanorozco @wassgha @aghassemi