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

special-case gifv #10

Open
forresto opened this issue Oct 21, 2015 · 4 comments
Open

special-case gifv #10

forresto opened this issue Oct 21, 2015 · 4 comments
Assignees
@forresto
Labels
enhancement

Comments

@forresto
Copy link
Contributor

Embedly's iframe is dumb (loads a 17mb gif poster for a <1mb mp4)... We can do better and output a <video> tag with mp4 and webm sources (just change .gifv to .mp4, .webm) ... still poster image h.jpg.
@forresto forresto self-assigned this Oct 21, 2015
@forresto
Copy link
Contributor Author

forresto commented Aug 2, 2016

Where would this plug in @bergie ?

@bergie
Copy link
Contributor

bergie commented Aug 2, 2016

@forresto probably in the iframe handler

@forresto
Copy link
Contributor Author

forresto commented Aug 2, 2016

Embedly (or Imgur) fixed the poster image issue. Still would be nice to work with video element instead of iframe.

http://embed.ly/docs/explore/extract?url=http%3A%2F%2Fi.imgur.com%2FaK4RY7D.gifv

@bergie
Copy link
Contributor

bergie commented Aug 2, 2016 •
edited
Loading

I think in general it might make sense to look at "un-embedlyizing" some known sources, to skip the middleman. So, direct video element for imgur gifv, direct youtube iframe, etc

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

@forresto forresto

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bergie @forresto