Skip to content
This repository has been archived by the owner on Oct 16, 2018. It is now read-only.

video.js version 5 compatibility pending? #12

Open
tweed opened this issue May 27, 2015 · 3 comments
Open

video.js version 5 compatibility pending? #12

tweed opened this issue May 27, 2015 · 3 comments

Comments

@tweed
Copy link

tweed commented May 27, 2015

Thank you greatly for the superb look - videojs-sublime-skin is a sensation with version 4.

Although new to video.js I'm enjoying the responsive features in video.js v5 -- and, although i hesitated before writing here, and also considered having a go at deducing how to mod/merge etc. to get videojs-sublime-skin to work with video.js v5, a bit of a play-around and I became kinda certain I'd ultimately fail.

So, here I am asking if/when it may b4 possible to have videojs-sublime-skin work with video.js v5?

in absolutely any event thank you very much!

@zakj
Copy link
Member

zakj commented May 28, 2015

Thanks for the kind words! I haven't been following the development of video.js v5, so I'm not sure how much work would go into updating this skin. I'd be happy to review a PR but I can't promise I'll have time to spare on a major effort in the near future.

@zanechua
Copy link

I have actually managed to complete one that works for version 5, however I don't think it has backward compatibility with version 4.

Do you still want me to do a PR?

Here's the codepen for version 5.
http://codepen.io/zanechua/pen/GozrNe

@commonpike
Copy link

commonpike commented Apr 14, 2016

That version seems to be pretty perfect as far as I can see !

Almost. It's pretty perfect at 10px. Increase the font size and the controls get out of line.

I bumped into this because I accidently loaded the stylesheet twice, causing IE to apply rules like font-size:1.8em twice on the same element (causing font-size to be 3.24em .. this seems like an ie bug).

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

4 participants