Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#402 closed enhancement (fixed)

nicer skin for video entries

Reported by: Sacha Owned by: Christopher Allan Webber
Priority: major Milestone: 0.3.0
Component: component1 Keywords:
Cc: Parent Tickets:

Description

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

Change History (10)

comment:1 by Sacha, 13 years ago

Owner: changed from somebody to Sacha
Status: newassigned

comment:2 by Christopher Allan Webber, 13 years ago

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???

comment:3 by Christopher Allan Webber, 13 years ago

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 %}
Last edited 13 years ago by Christopher Allan Webber (previous) (diff)

comment:4 by Sacha, 13 years ago

Owner: changed from Sacha to Christopher Allan Webber

comment:5 by Christopher Allan Webber, 13 years ago

Type: defectenhancement

comment:6 by Christopher Allan Webber, 13 years ago

Resolution: fixed
Status: assignedclosed

Looks soooooo awesome! Merged!

comment:7 by joar, 13 years ago

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.

in reply to:  7 ; comment:8 by Sacha, 13 years ago

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?

in reply to:  8 comment:9 by joar, 13 years ago

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.

comment:10 by joar, 13 years ago

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

Note: See TracTickets for help on using tickets.