#224773 - lightbox2 was displaying previously hidden select elements on end
authorStella Power
Fri, 22 Feb 2008 16:54:18 +0000 (16:54 +0000)
committerStella Power
Fri, 22 Feb 2008 16:54:18 +0000 (16:54 +0000)
js/lightbox.js
js/lightbox_lite.js

index 56db259..ff70a9a 100644 (file)
@@ -335,7 +335,7 @@ var Lightbox = {
   start: function(imageLink) {
 
     // replaces hideSelectBoxes() and hideFlash() calls in original lightbox2
-    $("select, embed, object").hide();
+    $("select[display!='none'], embed[display!='none'], object[display!='none']").hide();
     
     // stretch overlay to fill page and fade in
     var arrayPageSize = Lightbox.getPageSize();
index a82714a..f1563da 100644 (file)
@@ -212,7 +212,9 @@ function showLightbox(objLink) {
     // Hide select boxes as they will 'peek' through the image in IE
     selects = document.getElementsByTagName("select");
     for (i = 0; i != selects.length; i++) {
-      selects[i].style.visibility = "hidden";
+      if (selects[i].style.display != "none") {
+        selects[i].style.visibility = "hidden";
+      }
     }
 
     objLightbox.style.display = 'block';
@@ -253,7 +255,9 @@ function hideLightbox() {
   // make select boxes visible
   selects = document.getElementsByTagName("select");
   for (i = 0; i != selects.length; i++) {
-    selects[i].style.visibility = "visible";
+    if (selects[i].style.display != "none") {
+      selects[i].style.visibility = "visible";
+    }
   }
 
   // disable keypress listener