Fix vodkabears#131: Opera 12.1 bug with animations

WPDevHQ · May 24, 2015 · 7c95b6e · 7c95b6e
1 parent 6cd095d
commit 7c95b6e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/src/remodal.css b/src/remodal.css
@@ -24,9 +24,7 @@ html.remodal-is-locked {
   right: -5000px;
   bottom: -5000px;
   left: -5000px;
-}
 
-.remodal-overlay.remodal-is-closed {
   display: none;
 }
 
@@ -40,17 +38,14 @@ html.remodal-is-locked {
   bottom: 0;
   left: 0;
 
+  display: none;
   overflow: auto;
 
   text-align: center;
 
   -webkit-overflow-scrolling: touch;
 }
 
-.remodal-wrapper.remodal-is-closed {
-  display: none;
-}
-
 .remodal-wrapper:after {
   display: inline-block;
 

diff --git a/src/remodal.js b/src/remodal.js
@@ -385,7 +385,8 @@
     });
 
     instance.$bg.removeClass(instance.settings.modifier);
-    instance.$overlay.removeClass(instance.settings.modifier);
+    instance.$overlay.removeClass(instance.settings.modifier).hide();
+    instance.$wrapper.hide();
     unlockScreen();
     setState(instance, STATES.CLOSED, true);
   }
@@ -567,8 +568,8 @@
     current = remodal;
     lockScreen();
     remodal.$bg.addClass(remodal.settings.modifier);
-    remodal.$overlay.addClass(remodal.settings.modifier);
-    remodal.$wrapper.scrollTop(0);
+    remodal.$overlay.addClass(remodal.settings.modifier).show();
+    remodal.$wrapper.show().scrollTop(0);
 
     syncWithAnimation(
       function() {
@@ -610,7 +611,8 @@
 
       function() {
         remodal.$bg.removeClass(remodal.settings.modifier);
-        remodal.$overlay.removeClass(remodal.settings.modifier);
+        remodal.$overlay.removeClass(remodal.settings.modifier).hide();
+        remodal.$wrapper.hide();
         unlockScreen();
 
         setState(remodal, STATES.CLOSED, false, reason);