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

Convert YouTube shortcode and oEmbed to <amp-youtube> #5

Closed
mjangda opened this issue Sep 29, 2015 · 2 comments
Closed

Convert YouTube shortcode and oEmbed to <amp-youtube> #5

mjangda opened this issue Sep 29, 2015 · 2 comments
Labels
Enhancement New feature or improvement of an existing one
Milestone

Comments

@mjangda
Copy link
Contributor

mjangda commented Sep 29, 2015

See https://github.com/ampproject/amphtml/blob/master/extensions/amp-youtube/amp-youtube.md

@mjangda mjangda added the Enhancement New feature or improvement of an existing one label Sep 29, 2015
@mjangda mjangda added this to the Developer Preview milestone Sep 29, 2015
@mjangda mjangda modified the milestones: Developer Preview, v1 Oct 30, 2015
@corysimmons
Copy link

When I replace the thumbnail with a featured video it puts the video on top of the entire webpage (immediately inside <body>) whereas with just the featured image it puts it at the top of the content appropriately (after the post's header).

https://github.com/Automattic/amp-wp/blob/master/readme.md#featured-image

Update: just ended up executing the youtube shortcode instead. :(

@mjangda
Copy link
Contributor Author

mjangda commented Mar 8, 2016

@corysimmons Can you share the code you were using?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or improvement of an existing one
Projects
None yet
Development

No branches or pull requests

2 participants