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

Bookmarks are not appearing immediately #553

Open
0xifarouk opened this issue Jun 5, 2019 · 8 comments
Open

Bookmarks are not appearing immediately #553

0xifarouk opened this issue Jun 5, 2019 · 8 comments

Comments

@0xifarouk
Copy link

When adding a bookmark to a streaming video, it's not appearing immediately, I need to play the video for a couple of seconds to get it appears.
I am not sure of downloaded videos, but I think they have the same problem.

@allenhumphreys
Copy link
Collaborator

Do you mean "favorite"?

@0xifarouk
Copy link
Author

0xifarouk commented Jun 5, 2019

No, have a look here.

@allenhumphreys
Copy link
Collaborator

Sorry I was confused. Could you upload a brief gif or screen recording of the behavior by chance?

@allenhumphreys
Copy link
Collaborator

The delay has to do with generating a thumbnail image we use for the UI. It's likely way more pronounced when using when playing a streaming video compared to a downloaded video (generating a thumbnail will be dependent on internet connection as well). To make the UI snappier, the architecture would need to change somewhat to allow for a deferred delivery of a video thumbnail after the creation of the bookmark.

For reference:
Screen Shot 2019-06-05 at 9 31 15 PM

@0xifarouk
Copy link
Author

I still don't understand the relation between downloading the thumbnail and creating a bookmark? can't the thumbnail created after the bookmark?

@0xifarouk
Copy link
Author

0xifarouk commented Jun 6, 2019

I have uploaded a video here to show you what I exactly mean, it took about 14 seconds for the bookmark icon to appear in the timeline.

@allenhumphreys
Copy link
Collaborator

Yeah. This is a valid issue. The program flow seems to optimize for the downloaded videos, but it does work as implemented.

@insidegui
Copy link
Owner

Yeah we need to implement lazy loading of thumbnails for bookmarks.

Sent with GitHawk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants