-
Notifications
You must be signed in to change notification settings - Fork 932
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
feat: live_video_preview #2415
base: main
Are you sure you want to change the base?
feat: live_video_preview #2415
Conversation
Since |
hey Chris, thanks for looking at this. How would I use |
Sorry, I thought the code comment was referring to both js and server, so the difference its he video tag. I know the some video file types for me work just fine with live_img_preview, but it may not be as comprehensive. Let me explore this one some more. Thanks! |
ok, yeah the Hook can be removed indeed. I'll push a new commit removing that part |
ok, another thought I am having concerning this is maybe we just need a What do you think about that idea? |
Live Video Preview
For a project I am working on we needed a live video preview. After trying to implement it in our project it was far easier and made more sense to do that here.
So this PR adds:
live_img_preview