fix: body element is null when popup is initialized outside of it, cl…

…oses dimsemenov#305
hinchcliffz · Dec 4, 2013 · bcffd5e · bcffd5e
1 parent 7ddf368
commit bcffd5e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/dist/jquery.magnific-popup.js b/dist/jquery.magnific-popup.js
@@ -134,7 +134,6 @@ MagnificPopup.prototype = {
 		// We disable fixed positioned lightbox on devices that don't handle it nicely.
 		// If you know a better way of detecting this - let me know.
 		mfp.probablyMobile = (mfp.isAndroid || mfp.isIOS || /(Opera Mini)|Kindle|webOS|BlackBerry|(Opera Mobi)|(Windows Phone)|IEMobile/i.test(navigator.userAgent) );
-		_body = $(document.body);
 		_document = $(document);
 
 		mfp.popupsCache = {};
@@ -146,6 +145,10 @@ MagnificPopup.prototype = {
 	 */
 	open: function(data) {
 
+		if(!_body) {
+			_body = $(document.body);
+		}
+
 		var i;
 
 		if(data.isObj === false) { 
@@ -340,14 +343,11 @@ MagnificPopup.prototype = {
 
 		_mfpTrigger('BuildControls');
 
-
 		// remove scrollbar, add margin e.t.c
 		$('html').css(windowStyles);
 
 		// add everything to DOM
-		mfp.bgOverlay.add(mfp.wrap).prependTo( mfp.st.prependTo );
-
-
+		mfp.bgOverlay.add(mfp.wrap).prependTo( mfp.st.prependTo || _body );
 
 		// Save last focused element
 		mfp._lastFocusedEl = document.activeElement;
@@ -887,7 +887,7 @@ $.magnificPopup = {
 
 		removalDelay: 0,
 
-		prependTo: document.body,
+		prependTo: null,
 
 		fixedContentPos: 'auto',