Merge pull request #11705 from DanielRuf/fix/11704-reveal-enable-disa…

…ble-scroll-visible fix: enable and disable scroll if there is no visible reveal
foundation · Aug 12, 2019 · f77a4bc · f77a4bc
2 parents 86deccd + 2931b57
commit f77a4bc
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/js/foundation.reveal.js b/js/foundation.reveal.js
@@ -273,7 +273,9 @@ class Reveal extends Plugin {
       this.$element.trigger('closeme.zf.reveal', this.id);
     }
 
-    this._disableScroll();
+    if ($('.reveal:visible').length === 0) {
+      this._disableScroll();
+    }
 
     var _this = this;
 
@@ -446,7 +448,9 @@ class Reveal extends Plugin {
 
       _this.$element.attr('aria-hidden', true);
 
-      _this._enableScroll(scrollTop);
+      if ($('.reveal:visible').length  === 0) {
+        _this._enableScroll(scrollTop);
+      }
 
       /**
       * Fires when the modal is done closing.