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.

Sign up for GitHub

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

Fallback for iframe/audio/video #354

Closed
mjangda opened this issue Mar 28, 2016 · 2 comments
Closed

Fallback for iframe/audio/video #354

mjangda opened this issue Mar 28, 2016 · 2 comments
Labels
Enhancement New feature or improvement of an existing one P0 High priority

Comments

@mjangda
Copy link
Contributor

mjangda commented Mar 28, 2016

These AMP elements should have a fallback in case the media item fails to load (or is incompatible with AMP, e.g. non-https media).
This was referenced Mar 28, 2016
Closed
Closed
@mjangda mjangda added Enhancement New feature or improvement of an existing one P0 High priority labels Aug 18, 2016
@mjangda mjangda mentioned this issue Aug 18, 2016
Closed
@mjangda mjangda added this to the v0.4 milestone Sep 9, 2016
@coreymckrill coreymckrill mentioned this issue Sep 19, 2016
Closed
@coreymckrill coreymckrill mentioned this issue Sep 29, 2016
Closed
3 tasks
@westonruter westonruter modified the milestones: v0.4, v0.6, v0.6 (Q4 2017), v0.7 (Q1 2018) Nov 7, 2017
@westonruter
Copy link
Member

I'm not sure what we'd put in as a fallback. A link to the media?

@westonruter westonruter removed this from the v0.7 milestone Mar 7, 2018
@ThierryA ThierryA added Sprint 5 and removed Sprint 5 labels Mar 8, 2018
@westonruter
Copy link
Member

Fallbacks for iframes and media elements have been implemented in #1913 and #1861.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement New feature or improvement of an existing one P0 High priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mjangda @westonruter @ThierryA