From 53ba2919831c4f06bf94752a7aab69c8870b1e08 Mon Sep 17 00:00:00 2001 From: Vladimir Siljkovic Date: Sat, 8 Dec 2018 13:21:17 +0100 Subject: [PATCH] Breaking: Remove `backgroundactive` option Use `background:transparent` or `background:transparent, absolute:true` instead. --- jquery.popupoverlay.js | 33 ++------------------------------- 1 file changed, 2 insertions(+), 31 deletions(-) diff --git a/jquery.popupoverlay.js b/jquery.popupoverlay.js index c0ed4d2..3f7d73b 100644 --- a/jquery.popupoverlay.js +++ b/jquery.popupoverlay.js @@ -47,7 +47,6 @@ var $body = $('body'); var $wrapper; var options = $el.data('popupoptions'); - var css; bodymarginright = parseInt($body.css('margin-right'), 10); transitionsupport = document.body.style.webkitTransition !== undefined || @@ -61,12 +60,6 @@ options.scrolllock = false; } - if (options.backgroundactive) { - options.background = false; - options.blur = false; - options.scrolllock = false; - } - if (options.scrolllock) { // Calculate the browser's scrollbar width dynamically var parent; @@ -163,22 +156,14 @@ verticalAlign: 'middle' }); - css = { + $wrapper.css({ position: 'fixed', width: '100%', height: '100%', top: 0, left: 0, textAlign: 'center' - }; - - if(options.backgroundactive){ - css.position = 'absolute'; - css.height = '0'; - css.overflow = 'visible'; - } - - $wrapper.css(css); + }); // CSS vertical align helper $wrapper.append('