Addressing a problem that prevented the overlay functioning correctly…

… in Firefox
tommcfarlin · Jun 24, 2011 · 65f3091 · 65f3091
1 parent 041ff37
commit 65f3091
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/js/jquery.overlay.js b/js/jquery.overlay.js
@@ -6,7 +6,7 @@
  * Copyright 2011 Tom McFarlin, http://tommcfarlin.com, @moretom
  * Released under the MIT License
  *
- * http://moreco.de/jquery-simple-overlay
+ * http://moreco.de/simple-overlay
  */
 
 (function($) {
@@ -38,12 +38,19 @@
     $src.addClass('overlay-trigger');
 
     // create the overlay and add it to the dom
+    var iTop = 0;
+    if($.browser.mozilla && opts.container.toString() === 'body') { 
+      iTop = $('html').scrollTop();
+    } else {
+      iTop = $(opts.container).scrollTop();
+    } // end if/else
+
     var overlay = $('<div></div>')
       .addClass('overlay')
       .css({
         background: opts.color,
         opacity: opts.opacity,
-        top: opts.container.toString() === 'body' ? $(opts.container).scrollTop() : $(opts.container).offset().top,
+        top: opts.container.toString() === 'body' ? iTop : $(opts.container).offset().top,
         left: $(opts.container).offset().left,
         width: opts.container === 'body' ? '100%' : $(opts.container).width(),
         height: opts.container === 'body' ? '100%' : $(opts.container).height(),

diff --git a/js/jquery.overlay.min.js b/js/jquery.overlay.min.js