Opened 9 years ago

Closed 8 years ago

#375 closed defect (fixed)

Video Preload

Reported by: Jeremy Pope Owned by: joar
Priority: minor Milestone: 0.3.0
Component: component1 Keywords: bitesized
Cc: Parent Tickets:

Description (last modified by Christopher Allan Webber)

The HTML5 video tag has a few options on how a video is loaded on
page load. Currently, MediaGoblin uses ``preload="auto"`` which
causes the whole video to be downloaded on the end users side, even
if they don't actually play the video.
The ``preload="metadata"`` option will only load the metadata and
first frame of the video on page load. This can help site
navigation quite a bit during low-bandwidth times (such as through
3G on phone's wifi hotspot).
I have edited the
``mediagoblin/templates/mediagoblin/media_displays/video.html``
file on my MG instance to have the 'metadata' option. A couple of
examples: `http://url.jpope.org/9a <http://url.jpope.org/9a>`_ and
`http://url.jpope.org/99 <http://url.jpope.org/99>`_



Subtickets

Change History (7)

comment:1 Changed 9 years ago by Jeremy Pope

And I forgot to drop this link for reference sake ;)
`http://www.w3schools.com/html5/att\_video\_preload.asp <http://www.w3schools.com/html5/att_video_preload.asp>`_



comment:2 Changed 9 years ago by Will Kahn-Greene

The original url for this bug was http://bugs.foocorp.net/issues/734 .

comment:3 Changed 8 years ago by Christopher Allan Webber

Component: component1
Description: modified (diff)
Keywords: bitesized added
Milestone: 0.2.2

This is an easy task for a new contributor.

comment:4 Changed 8 years ago by Christopher Allan Webber

Also jpope's "theme" changed, so that example went away... should be clear enough what to do from the description of this bug I think, though.

comment:5 Changed 8 years ago by joar

Owner: set to joar
Status: acceptedassigned

This has been solved in a commit on my audio+sniffing branch, review and merge into master is pending.

https://github.com/jwandborg/mediagoblin/commit/4f4f2531ad5207d074027519b881bf485a249b44#L7R27

comment:6 Changed 8 years ago by Sacha

I fixed this bug here:
https://gitorious.org/~chemhacker/mediagoblin/chemhackers-mediagoblin/commits/bug375_video_preload

...and Chris already pushed my fix to the master branch. (collisions in sprints :) )

comment:7 Changed 8 years ago by Christopher Allan Webber

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.