Error javascript al ocultar capa

[ferent]

He modificado un script de un usuario de foros del web, pero al ejecutarlo me da un error y no sé como solucionarlo, este es el script:

Código PHP:

  <script LANGUAGE="javascript" type="text/javascript"> 
var capa 
var capa2 
function cambio(idCapa){ 
 
 if (document.layers) capa = eval("document." + idCapa); 
 if (document.all) capa = eval(idCapa + ".style"); 
 if (document.getElementById) capa = eval('document.getElementById("' + idCapa + '").style'); 
  
 
 if ((capa.visibility == "hidden") || (capa.visibility == "hide")){ 
  capa.visibility = (document.layers) ? "show" : "visible"; 
 
 }else{ 
  capa.visibility = (document.layers) ? "hide" : "hidden"; 
 
 } 
  
 if (document.layers) capa2 = eval("document." + idCapa); 
 if (document.all) capa2 = eval(idCapa + ".style"); 
 if (document.getElementById) capa2 = eval('document.getElementById("' + idCapa + '").style'); 
  
 if ((capa2.display == "none") || (capa2.display == "hide")){ 
  capa2.display = (document.layers) ? "show" : "inherit"; 
 
 }else{ 
  capa2.display = (document.layers) ? "hide" : "none"; 
 
 } 
  
} 
//--> 
</script> 
 
---------------------------------------- 
<a href="#" onclick="cambio('capatemp')"><img src="boton_usuario.jpg" alt="nuevo usuario" width="123" height="33" border="0" /></a> 
<div id="capatemp" style='position:relative; visibility:hidden; display:none'></div>  
  

El error que dice el explorer es que no se puede obtener la propiedad display. Argumento no válido.

¿Saben como resolverlo?

PD: Por cierto, en firefox se ejecuta sin problemas :)

[tunait]

Hola ferent,

prueba así

Código:

<script LANGUAGE="javascript" type="text/javascript">
var capa
var capa2
function cambio(idCapa){

capa =document.getElementById(idCapa).style;
 

 if (capa.visibility == "hidden"){
  capa.visibility =  "visible";

 }else{
  capa.visibility = "hidden";

 }
 
capa2 = document.getElementById(idCapa).style;
 
 if (capa2.display == "none") {
  capa2.display = "inherit";

 }else{
  capa2.display =  "none";

 }
 
}
//-->
</script>

Ya me contarás

saludos

[ferent]

Gracias por tu respuesta Tunait, me sigue dando error en la linea 89 que contiene:

capa2.display = "inherit";

¿alguna idea?

PD: ¡Así todo tu código está mucho más optimizado que el que usaba!

[tunait]

Hum .. inherit no será un valor posible para la propiedad display. Cambia inherit por block

¡Así todo tu código está mucho más optimizado que el que usaba!

Bueno, sólo eliminé las partes en donde se comprueba si es un navegador muy antíguo y realmente no vale la pena mantener esos condicionales.

Prueba a ver qué tal le parece block

Un saludo

Edito: por cierto que aún se puede optimizar más si quieres

[caricatos]

Hola:

Para display, inherit no parece un valor lógico (nunca lo he usado)... he visto en algunos códigos de JavierB poner la cadena vacía ("")... personalmente prefiero darle un valor más definido (inline/block/none)

Coincido con tunait (¡Hola !) en que puede conseguirse un código más optimizado, incluso buscando otros mensajes en estos mismos foros... (estos días participé en una cuestión similar)

Saludos

[ferent]

Pues no va tampoco, les pido que accedan a la web para que puedan ver en vivo el fallo (no testado en explorer 7, solo en 6).

En la web tienen que clicar en gestion de usuarios (abre un iframe) y clicar en el boton Nuevo usuario:

eliminada

CORRIJO: Fallo mio en esta ocasión, ya funciona!! gracias