-
Notifications
You must be signed in to change notification settings - Fork 22
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
Feature/video #9
Conversation
Thanks for being so active on this project! |
Btw, the video files have made this repository super heavy. What do you say to using some already-hosted video files instead? |
this is a new feature developed by me with a pull request here fat#66 I tested it on multiple browsers and mobile too and it works well, so I decide to merge on master project. I can upload tomorrow other videos (<5mb) or I can try to link to an extarnal resource |
I've uploaded smaller videos now as a part of #12. |
Many thanks |
@spinningarrow could you take a look at this? I've been trying to create a solution, but it's been a pretty hard challenge to solve. fat#77 |
@ElegantSudo could you give me a codepen or something that demonstrates the problem? I mostly need it for testing. |
@spinningarrow of course! http://codepen.io/yipyop/pen/RRpOKQ I've outlined the problem and given an example. |
Looks good! Thanks a lot; I'll take a crack at it. |
@ElegantSudo take a look at #18 when you can! |
Add new feature: zoom on video too