dominic-p / videojs-resolution-selector

Adds a resolution selector button to Video.js to allow users to manually adjust the video quality.
MIT License
88 stars 31 forks source link

plugin "resolutionSelector" does not exist,Uncaught TypeError: Cannot read property 'extend' of undefined #41

Open Syedwaheed opened 7 years ago

Syedwaheed commented 7 years ago

I m using this feature but getting two errors and even cant play video when embedding this feature. Any help would be appreciated Thanks. capture capture1

dominic-p commented 7 years ago

It's most likely a compatibility issue with a recent version of Video.js. I haven't had time to update this plugin in a while, so it's broken on modern versions of Video.js.

It looks like some other people have switched to this plugin: https://github.com/kmoskwiak/videojs-resolution-switcher

I don't know anything about it, but you might give it a try.

Syedwaheed commented 7 years ago

Thanks for your reply i have gone through that before this one but that's too have some exceptions.. so i thought to try this one.

On 26 Oct 2017 9:52 p.m., "Dominic" notifications@github.com wrote:

It's most likely a compatibility issue with a recent version of Video.js. I haven't had time to update this plugin in a while, so it's broken on modern versions of Video.js.

It looks like some other people have switched to this plugin: https://github.com/kmoskwiak/videojs-resolution-switcher

I don't know anything about it, but you might give it a try.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dominic-p/videojs-resolution-selector/issues/41#issuecomment-339729614, or mute the thread https://github.com/notifications/unsubscribe-auth/ANa3yTeXE--pLl3TSHs7QfLuZGZjHp7Gks5swLjMgaJpZM4QHKkc .

dominic-p commented 7 years ago

Gotcha. Yeah, eventually I plan to update this, but I'm not sure when that will be. But, I'll go through and close all of these issues when I do, so you should be notified.

Syedwaheed commented 7 years ago

Ok thanks i will be waiting for this.

On 27 Oct 2017 12:53 a.m., "Dominic" notifications@github.com wrote:

Gotcha. Yeah, eventually I plan to update this, but I'm not sure when that will be. But, I'll go through and close all of these issues when I do, so you should be notified.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dominic-p/videojs-resolution-selector/issues/41#issuecomment-339781876, or mute the thread https://github.com/notifications/unsubscribe-auth/ANa3ydUNq2ca0JErl9le5VXHRdJYAYo6ks5swONUgaJpZM4QHKkc .

lonesomewalker commented 7 years ago

Hi Dominic, your suggested switch doesn't really look promising. Suggests to use an older video.js version. Seriously, then i will stick with yours, proven to work :-)

dominic-p commented 7 years ago

I didn't see that. It looks like not a lot of development is happening over there either. You can also experiment with the 5.x compatibility pull request #37 to see if that works for you.