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

feature: Add pre-downloading of video to prevent buffering like on youtube #1275

Closed
ashutosh017 opened this issue Sep 19, 2024 · 7 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@ashutosh017
Copy link
Contributor

I think currently app.100xdevs doesn't pre-download video like on youtube, hence it buffers a lot.

no-predownloading

predownloading-example

@ashutosh017 ashutosh017 added the enhancement New feature or request label Sep 19, 2024
@CuriousCoder00
Copy link
Contributor

I did some research on it and it can be done with defining preloading that suggests to the browser whether or not the video data should begin downloading as soon as the video element is loaded.
Need to do a few tests. @devsargam can you assign it to me?

@devsargam
Copy link
Collaborator

Cool guys

@devsargam
Copy link
Collaborator

Guys @ashutosh017 @CuriousCoder00 can u guys take it from here?

@CuriousCoder00
Copy link
Contributor

@devsargam

Tried playing with video player and added a longer video for better testing

Untitled.video.-.Made.with.Clipchamp.2.mp4

@devsargam
Copy link
Collaborator

Cool want to see the changes can u pr it?

@CuriousCoder00
Copy link
Contributor

CuriousCoder00 commented Sep 20, 2024

@devsargam I'm having a bit issue in raising the PR, Rather then issue maybe a mistake or something

@CuriousCoder00
Copy link
Contributor

Hey @devsargam Upon further investigation, I found that video preloading does occur. However, it is not easily noticeable for longer videos. This behavior is also observed with YouTube videos. Please see the attached screenshot for reference.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants