Merge pull request mediaelement#278 from erktime/5e00723e0955343093e0…

…125ec2d716b827bd4f9b Found a few issues with your last couple of changes to fullscreen. Perhaps they'll help.
gavinlynch · Oct 26, 2011 · 8660028 · 8660028
2 parents 1b45f93 + dd63a2d
commit 8660028
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/src/js/mep-feature-fullscreen.js b/src/js/mep-feature-fullscreen.js
@@ -119,10 +119,13 @@
 				.width('100%')
 				.height('100%');
 				//.css({position: 'fixed', left: 0, top: 0, right: 0, bottom: 0, overflow: 'hidden', width: '100%', height: '100%', 'z-index': 1000});				
-			setTimeout(function() {
-				t.container.css({width: '100%', height: '100%'});
-			}, 500);
-			//console.log('fullscreen', t.container.width());
+
+			// Only needed for safari 5.1 native full screen, can cause display issues elsewhere
+			if (mejs.MediaFeatures.hasTrueNativeFullScreen) {
+				setTimeout(function() {
+					t.container.css({width: '100%', height: '100%'});
+				}, 500);
+			}
 
 			if (t.pluginType === 'native') {
 				t.$media
@@ -171,8 +174,7 @@
 			t.container
 				.removeClass('mejs-container-fullscreen')
 				.width(normalWidth)
-				.height(normalHeight)
-				.css('z-index', 1);
+				.height(normalHeight);
 				//.css({position: '', left: '', top: '', right: '', bottom: '', overflow: 'inherit', width: normalWidth + 'px', height: normalHeight + 'px', 'z-index': 1});
 
 			if (t.pluginType === 'native') {
@@ -200,4 +202,4 @@
 		}	
 	});
 
-})(mejs.$);
+})(mejs.$);