Skip to content
This repository has been archived by the owner on Jan 12, 2019. It is now read-only.

vjs-has-started not updated with 4.6 #65

Closed
CodingFabian opened this issue May 21, 2014 · 20 comments
Closed

vjs-has-started not updated with 4.6 #65

CodingFabian opened this issue May 21, 2014 · 20 comments

Comments

@CodingFabian
Copy link

Hi,
it seems that some refactoring between 4.5 and 4.6 broke the vjs-has-started / related callbacks. I can click the button and the hls video plays, but the css class is not applied.
if i manually hack it in the dom, the button disappears and the controlbar appreas correctly.

@gkatsev
Copy link
Member

gkatsev commented May 21, 2014

We'll look into this once we get the hls-tech (#62) work merged in.

@CodingFabian
Copy link
Author

@gkatsev i can still observe the same after upgrading to the merged #62

@CodingFabian
Copy link
Author

when I downgrade videojs to 4.5.1 then it works for me. looks like others have similar problems (videojs/video.js#1226). I consider it more likely to be a videojs rather than a hls-contrib issue, but would appreciate you looking into it anyway

@gkatsev
Copy link
Member

gkatsev commented May 22, 2014

We are looking into it. It is most likely a video-js-swf issue.

@gkatsev
Copy link
Member

gkatsev commented May 23, 2014

This should be fixed in in #66.

@mkilling
Copy link

The video controls still don't show up for me after pressing the play button (Google Chrome). My setup:

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

@mkilling you need videojs/video-js-swf#93 as well

@mkilling
Copy link

@gkatsev Wasn't videojs/video-js-swf#93 already merged into videojs/video-js-swf master?

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

@mkilling so it was. I forgot. Hm... it was definitely working for us yesterday...
Is it giving you any errors? Also, are you loading it properly as a tech now?

@mkilling
Copy link

No errors in the console...

<video id="video-player" class="video-js vjs-default-skin" controls preload="auto" width="620px" height="465px" poster="https://my_thumbnail">
    <source type="application/x-mpegURL" src="http://s3.amazonaws.com/my_bucket/my_video">
</video>

I initialize the player like this:

videojs("video-player", { playbackRates: [1, 1.5, 2, 3], techOrder: ['html5', 'hls', 'flash'] });

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

Is it playing otherwise?

@mkilling
Copy link

Yes, the video is playing. Only the large play button in the top left corner does not disappear and the video controls don't show up. It works with the same setup when pointing it to an mp4 file instead of the m3u8

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

Hm... I guess we didn't fix this it fully. Thanks for reporting.

@mkilling
Copy link

Turns out I didn't specify the right .swf file. It works for me now. Sorry for the confusion.

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

@mkilling No worries. So, it works as expected now?

@mkilling
Copy link

Yes it does. Controls show up and everything is fine ;-)

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

Awesome, thanks for verifying!

@CodingFabian
Copy link
Author

I can verify it is working here as well. Any chance to get all the stuff merged and released? Right now our next production deployment will contain unmerged and unrelease changes and I want to avoid maintenance nightmares :-) thanks!

@gkatsev
Copy link
Member

gkatsev commented May 25, 2014

It will get merged in this week. Hopefully, released this week as well.

@gkatsev
Copy link
Member

gkatsev commented Jun 6, 2014

Updates have been released. Closing now as it should be fixed, but feel free to reopen and comment if there are still problems.

@gkatsev gkatsev closed this as completed Jun 6, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants