Incompatible with new version 3 of video-js? [#1406422]

Comment #1

lightsurge CreditAttribution: lightsurge commented 16 January 2012 at 12:19

This is probably to be considered a duplicate of #1402846: Compatibility with VideoJS v. 3 (which I didn't spot because of the version)

I don't know if you want to leave this open for a patch backport.

Log in or register to post comments

Comment #2

Jorrit CreditAttribution: Jorrit commented 26 January 2012 at 20:23

Status:

Active

» Postponed (maintainer needs more info)

Please test tomorrows 6.x-1.x dev release to check if it works with VideoJS version 3.

Log in or register to post comments

Comment #3

lightsurge CreditAttribution: lightsurge commented 27 January 2012 at 10:46

This seemed to work fine on Chrome. Only one problem, I'm using it with Video module, and the video thumbnail didn't display, just got a black background, but that might be an issue for me to put in video.module's queue.

IE8 was another story however, it hanged the javascript interpreter. As usual the error message given was fairly useless, apart from pointing to this point in video-js code:

this.box.style.width = this.width() + "px";

in the "VideoJS Box - Holds all elements" section.

I suppose it must be something to do with the Flash fallback not working correctly, since that's where IE8 differs in terms of not being HTML5 happy.

On that subject, any plans to add in the Flash fallback player to 6.x? Maybe that would just fix it.

Log in or register to post comments

Comment #4

bradjones1

English

CreditAttribution: bradjones1 commented 27 January 2012 at 20:47

@Jorrit - mind looking at the D7 companion #1402846: Compatibility with VideoJS v. 3 and giving me your feedback?

Log in or register to post comments

Comment #5

Jorrit CreditAttribution: Jorrit commented 29 January 2012 at 12:16

I did not realize that the new VideoJS version brings its own Flash fall back player. I think it is best to create new 2.0 versions for both Drupal 6 and 7 that only support the new VideoJS version, because then a lot of code can be removed.