YTAPIReady is triggered also on mobile

pupunzi · Jun 17, 2016 · 944032f · 944032f
1 parent f10a7d0
commit 944032f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/dist/jquery.mb.YTPlayer.js b/dist/jquery.mb.YTPlayer.js
@@ -219,11 +219,6 @@ var getYTPVideoID = function( url ) {
 					YTPlayer.isPlayer = true;
 				}
 
-				if( jQuery.browser.mobile && !YTPlayer.canPlayOnMobile ) {
-					$YTPlayer.remove();
-					return;
-				}
-
 				var overlay = jQuery( "<div/>" ).css( {
 					position: "absolute",
 					top: 0,
@@ -320,6 +315,11 @@ var getYTPVideoID = function( url ) {
 					}, 100 )
 				}
 
+				if( jQuery.browser.mobile && !YTPlayer.canPlayOnMobile ) {
+					$YTPlayer.remove();
+					return;
+				}
+
 				jQuery( document ).on( "YTAPIReady", function() {
 					if( ( YTPlayer.isBackground && ytp.backgroundIsInited ) || YTPlayer.isInit ) return;
 					if( YTPlayer.isBackground ) {
@@ -336,6 +336,7 @@ var getYTPVideoID = function( url ) {
 
 						YTPlayer.isInit = true;
 
+
 						//if is mobile && isPlayer fallback to the default YT player
 						if( jQuery.browser.mobile && YTPlayer.canPlayOnMobile ) {
 							// Try to adjust the player dimention

diff --git a/src/jquery.mb.YTPlayer.src.js b/src/jquery.mb.YTPlayer.src.js
@@ -219,11 +219,6 @@ var getYTPVideoID = function( url ) {
 					YTPlayer.isPlayer = true;
 				}
 
-				if( jQuery.browser.mobile && !YTPlayer.canPlayOnMobile ) {
-					$YTPlayer.remove();
-					return;
-				}
-
 				var overlay = jQuery( "<div/>" ).css( {
 					position: "absolute",
 					top: 0,
@@ -320,6 +315,11 @@ var getYTPVideoID = function( url ) {
 					}, 100 )
 				}
 
+				if( jQuery.browser.mobile && !YTPlayer.canPlayOnMobile ) {
+					$YTPlayer.remove();
+					return;
+				}
+
 				jQuery( document ).on( "YTAPIReady", function() {
 					if( ( YTPlayer.isBackground && ytp.backgroundIsInited ) || YTPlayer.isInit ) return;
 					if( YTPlayer.isBackground ) {