WWDC icon indicating copy to clipboard operation
WWDC copied to clipboard

Bookmarks are not appearing immediately

Open 0xifarouk opened this issue 6 years ago • 8 comments

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.

0xifarouk avatar Jun 05 '19 20:06 0xifarouk

Do you mean "favorite"?

allenhumphreys avatar Jun 05 '19 20:06 allenhumphreys

No, have a look here.

0xifarouk avatar Jun 05 '19 23:06 0xifarouk

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

allenhumphreys avatar Jun 06 '19 01:06 allenhumphreys

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

allenhumphreys avatar Jun 06 '19 02:06 allenhumphreys

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

0xifarouk avatar Jun 06 '19 15:06 0xifarouk

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.

0xifarouk avatar Jun 06 '19 15:06 0xifarouk

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

allenhumphreys avatar Jun 06 '19 18:06 allenhumphreys

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

Sent with GitHawk

insidegui avatar Jun 06 '19 18:06 insidegui