nicer skin for video entries

[Sacha]

Current skin is boring, video.js is probably the way to go for this (see videojs.com).

[Christopher Allan Webber]

So the steps that are necessary to do this:

• Add video.js to extlib/ ... make sure it includes license files of some sort
• Symlink into mediagoblin/static/js/ with *relative* paths, probably like so

  cd mediagoblin/static/js/extlib/
  ln -s ../../../../video.js/video.js ./  # or whatever
  

• Include that via request.staticdirect in the mediagoblin_head jinja2 template block

  {% block mediagoblin_head %}
    <script type="text/javascript"
            src="{{ request.staticdirect('/js/extlib/video.js') }}"></script>
  {% endblock %}
  

• profit???

[Christopher Allan Webber]

Actually, to include the content of the previous block:

{% block mediagoblin_head %}
  {{ super() }}
  <script type="text/javascript"
          src="{{ request.staticdirect('/js/extlib/video.js') }}"></script>
{% endblock %}

[Sacha]

the fix is here:
​https://gitorious.org/~chemhacker/mediagoblin/chemhackers-mediagoblin/commits/bug402_nicer_skin_for_video

tested and is working with mp4 uploads

[Christopher Allan Webber]

Looks soooooo awesome! Merged!

[joar]

It seems the native controls are still visible beneath the video-js controls, perhaps we should remove the controls attribute from the <video> tag too.

[Sacha]

Replying to joar:

It seems the native controls are still visible beneath the video-js controls, perhaps we should remove the controls attribute from the <video> tag too.

can you tell me what browser you're seeing this in?

[joar]

Replying to chemhacker:

Replying to joar:

It seems the native controls are still visible beneath the video-js controls, perhaps we should remove the controls attribute from the <video> tag too.

can you tell me what browser you're seeing this in?

Google Chrome 19.0.1068.0 dev linux x86_64.

Iceweasel looks good.

[joar]

It seems this is an issue specific to google-chrome-unstable. Case closed closed :)