Partially complete addition of Vevo support #3

Merged
merged 1 commit into from Mar 25, 2012

2 participants

@AlecRust

Adding Vevo (http://www.vevo.com/) support to this script would be very useful, since a lot of Vevo content isn't embeddable when viewed through the embedded YouTube player.

I've given it a go, but had some trouble obtaining the video ID via Regex.

@AlecRust AlecRust Partially complete addition of Vevo support. Had a little trouble wit…
…h Regex getting the video ID, hopefully that's easy to fix.
88b6238
@AlecRust AlecRust commented on the diff Mar 23, 2012
js/jqvideobox.js
@@ -134,6 +139,9 @@ jQuery( function ($) {
else if (href.match(/youtu\.be/i)) {
type = 'youtu.be';
}
+ else if (href.match(/vevo\.com\/watch/i)) {

Regex issue here, I believe. Example Vevo URL: http://www.vevo.com/watch/maverick-sabre/no-one/GBUV71102090

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@emposha emposha merged commit e9b4859 into emposha:master Mar 25, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment