Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Jhurliman youtube #215

Merged
merged 3 commits into from Aug 13, 2012

Conversation

Projects
None yet
1 participant
Owner

heff commented Aug 13, 2012

Manually brining in jhurliman's youtube code. Some missing elements need to brought in without affecting core source.

heff added a commit that referenced this pull request Aug 13, 2012

@heff heff merged commit 8eba022 into videojs:master Aug 13, 2012

d3mcfadden added a commit to d3mcfadden/video-js that referenced this pull request Sep 14, 2012

Fix build script
When pull request #215 (heff/jhurliman-youtube) was merged it seems to have broke
the build script (build.sh). There is two issues this resolves:

1. video-js.swf was moved under tech and not updated in the build
2. html5.js does not have a leading linebreak in the file. When this file
is concat'ed on it results in a javascript error.

```
// _V_.Cue = _V_.Component.extend({
//   init: function(player, options){
//     this._super(player, options);
//   }
// });/* HTML5 Playback Technology - Wrapper for HTML5 Media API
================================================================================ */
```

@d3mcfadden d3mcfadden referenced this pull request Sep 14, 2012

Closed

Fix build script #226

@heff heff deleted the heff:jhurliman-youtube branch Jul 2, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment