Add workaround to fix #929 by manually setting the display to block i…

…nside the complete animation callback
ProjectMirador · Jun 6, 2016 · 5d4c8cb · 5d4c8cb
1 parent 76a35e7
commit 5d4c8cb
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 4 deletions.
diff --git a/js/src/viewer/workspacePanel.js b/js/src/viewer/workspacePanel.js
@@ -94,7 +94,15 @@
     },
 
     show: function() {
-      jQuery(this.element).show({effect: "fade", duration: 160, easing: "easeInCubic"});
+      var onComplete = function () {
+        // Under firefox $.show() used under display:none iframe does not change the display.
+        // This is workaround for https://github.com/IIIF/mirador/issues/929
+        jQuery(this).css('display', 'block');
+      };
+
+      jQuery(this.element).show({
+        effect: "fade", duration: 160, easing: "easeInCubic", complete: onComplete
+      });
     },
 
     onPanelVisible: function(_, stateValue) {

diff --git a/js/src/widgets/bookView.js b/js/src/widgets/bookView.js
@@ -216,7 +216,13 @@ bindEvents: function() {
     },
 
     show: function() {
-      jQuery(this.element).show({effect: "fade", duration: 300, easing: "easeInCubic"});
+      jQuery(this.element).show({
+        effect: "fade", duration: 300, easing: "easeInCubic", complete: function () {
+          // Under firefox $.show() used under display:none iframe does not change the display.
+          // This is workaround for https://github.com/IIIF/mirador/issues/929
+          jQuery(this).css('display', 'block');
+        }
+      });
     },
 
     adjustWidth: function(className, hasClass) {

diff --git a/js/src/widgets/imageView.js b/js/src/widgets/imageView.js
@@ -340,7 +340,13 @@
     },
 
     show: function() {
-      jQuery(this.element).show({effect: "fade", duration: 300, easing: "easeInCubic"});
+      jQuery(this.element).show({
+        effect: "fade", duration: 300, easing: "easeInCubic", complete: function () {
+          // Under firefox $.show() used under display:none iframe does not change the display.
+          // This is workaround for https://github.com/IIIF/mirador/issues/929
+          jQuery(this).css('display', 'block');
+        }
+      });
     },
 
     adjustWidth: function(className, hasClass) {

diff --git a/js/src/widgets/thumbnailsView.js b/js/src/widgets/thumbnailsView.js
@@ -108,7 +108,11 @@
     bindEvents: function() {
       var _this = this;
       _this.element.find('img').on('load', function() {
-        jQuery(this).hide().fadeIn(750);
+        jQuery(this).hide().fadeIn(750,function(){
+          // Under firefox $.show() used under display:none iframe does not change the display.
+          // This is workaround for https://github.com/IIIF/mirador/issues/929
+          jQuery(this).css('display', 'block');
+        });
       });
 
       jQuery(_this.element).scroll(function() {
@@ -199,6 +203,9 @@
         duration: 300,
         easing: "easeInCubic",
         complete: function() {
+          // Under firefox $.show() used under display:none iframe does not change the display.
+          // This is workaround for https://github.com/IIIF/mirador/issues/929
+          jQuery(this).css('display', 'block');
           _this.loadImages();
         }
       });