Browse files

Allowed fullscreening of HTML5 video by default.

  • Loading branch information...
1 parent 2c333e4 commit 960e7bc8bbd00a6552e9428517df952a31debb7b jackmoore committed Oct 7, 2012
Showing with 4 additions and 1 deletion.
  1. +4 −1 colorbox/jquery.colorbox.js
@@ -736,7 +736,10 @@
src: settings.href,
name: (new Date()).getTime(), // give the iframe a unique name to prevent caching
- 'class': prefix + 'Iframe'
+ 'class': prefix + 'Iframe',
+ allowFullScreen : true, // allow HTML5 video to go fullscreen
+ webkitAllowFullScreen : true,
+ mozallowfullscreen : true
.one('load', complete)
.one(event_purge, function () {

2 comments on commit 960e7bc

that sounds fuuuucking coool :-)
will that be in the next release 1.3.21?


jackmoore replied Oct 8, 2012

Yes, but I'm not sure how useful it is. It only applies when someone is iframing a HTML5 video, and IE doesn't support it (not sure about IE10). It's already something that can be done using the onComplete callback (#260). I told that user I didn't want it as a default, but on second thought it's probably for the best.

Please sign in to comment.