Vincent Petry

Results 514 comments of Vincent Petry

Ok managed to reproduce it somehow, add this to the server: ```diff diff --git a/lib/private/legacy/files.php b/lib/private/legacy/files.php index 8529446a0b..5df168d0f4 100644 --- a/lib/private/legacy/files.php +++ b/lib/private/legacy/files.php @@ -100,6 +100,7 @@ class OC_Files {...

suggestion: if the video player has an event that signals that playback has started, use that one to clear the spinner

doesn't seem to happen any more with 1.0.0RC1

hmm still happening but without the above error. I had to wait a long time for the first video in the list to load while the preload was busy with...

note: this was observed while testing https://github.com/owncloud/files_mediaviewer/pull/36 in public view mode

did you fix this in #36 ? I didn't retest it. else can be scheduled later

I guess the problem is mostly that one needs to find a keyframe to be able to render video data. For some reason it seems the HTML5 component is trying...

just now I have a case where it seems to download the full video (66mb) even though I did not play it

I just found this article: https://developers.google.com/web/fundamentals/performance/lazy-loading-guidance/images-and-video/ it seems that Chrome defaults to either preloading metadata or a few seconds of video, but not the full one. we could try setting...

possible fix: https://github.com/owncloud/files_mediaviewer/pull/70