Skip to content

Commit

Permalink
Fixed a minification issue with the fullscreen event. fixes #1282. cl…
Browse files Browse the repository at this point in the history
…oses #1286
  • Loading branch information
heff committed Jun 12, 2014
1 parent 39ae517 commit 3720a2d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ CHANGELOG

## HEAD (Unreleased)
* Updated to version 4.4.1 of the SWF ([view](https://github.com/videojs/video.js/pull/1285))
* Fixed a minification issue with the fullscreen event. fixes #1282 ([view](https://github.com/videojs/video.js/pull/1286))

--------------------

Expand Down
4 changes: 2 additions & 2 deletions src/js/player.js
Original file line number Diff line number Diff line change
Expand Up @@ -922,12 +922,12 @@ vjs.Player.prototype.requestFullscreen = function(){
// when cancelling fullscreen. Otherwise if there's multiple
// players on a page, they would all be reacting to the same fullscreen
// events
vjs.on(document, fsApi.fullscreenchange, vjs.bind(this, function(e){
vjs.on(document, fsApi['fullscreenchange'], vjs.bind(this, function(e){
this.isFullscreen(document[fsApi.fullscreenElement]);

// If cancelling fullscreen, remove event listener.
if (this.isFullscreen() === false) {
vjs.off(document, fsApi.fullscreenchange, arguments.callee);
vjs.off(document, fsApi['fullscreenchange'], arguments.callee);
}

this.trigger('fullscreenchange');
Expand Down

0 comments on commit 3720a2d

Please sign in to comment.