[SOLUCIONADO] Habilitar campos de un form al apretar un radio button

[WorseMzg]

Buneas a todos, primero soy nuevo pero siempre me guio por esta pagina asi que gracias de antemano

Estoy tratando de habilitar y deshabilitar campos al apretar un radio, según lo que entiendo el evento on click me tiene que llamar a una funcion. y es lo que trato de hacer pero no me ejecuta. aqui dejo el codigo. de antemano gracias.

Código:

<form enctype="multipart/form-data" method="post" name="formNoticias" action="">
                <table width="700" border="0" cellpadding="0" cellspacing="0" class="tbNuevaNoticia">
                 
                  <tr>
                    <td class="tituloCampo">¿Desea agregar una galeria de imagenes?</td>
                    <td class="inputCampo"><input type = "radio" name = "galeria" id = "galeria" value = "si" onclick="habilitarGaleria(si)"/>Si -<input type = "radio" name = "galeria" id = "galeria" value = "no" onclick="habilitarGaleria(no)" cheked="cheked" />No</td>
                  </tr>
                  <tr>
                    <td name="galeria" class="tituloCampo">Titulo de la galeria</td>
                    <td name="galeria" class="inputCampo"><input name="NombreGal" type="name" /></td>
                  </tr>
                  <tr>
                    <td name="galeria" class="tituloCampo" style="visibility: hidden;">Seleccione las imagenes:</td>
                    <td name="galeria" class="inputCampo" style="visibility: hidden;"><input name="uploadGaleria[]" type="file" multiple /></td>
                   </table>
            </form>

y aqui el javascript

Código:

function habilitarGaleria(tipo){

  if(tipo == "si")
  {
    document.formNoticias.galeria.style.visibility="visible";
  }
  if (tipo == "no")
  {
    document.formNoticias.galeria.style.visibility="hidden";
  }

}

me imagino que tendre varios errores de sintaxis, pero mas que nada me preocupa la logica de lo que estoy haciendo..


Se agradece

[PHPeros]

Tienes que poner las comillas, se trata de un string, no una variable:

Código:

onclick="habilitarGaleria('no')"

Saludos

[WorseMzg]

Iniciado por PHPeros

Tienes que poner las comillas, se trata de un string, no una variable:

Código:

onclick="habilitarGaleria('no')"

Saludos

sep eso ya lo corregi pero sigue sin ingresar a la funcion...

[PHPeros]

Y esto no existe:

cheked="cheked"

Por cierto. ¿Qué errores te lanza la consola?

[WorseMzg]

Iniciado por PHPeros

Y esto no existe:

cheked="cheked"

Por cierto. ¿Qué errores te lanza la consola?

checked es para definir que radio va a estar seleccionado por defecto

lo probe y eso funciona dependiendo de donde lo coloque es cual va a estar seleccionado si el si o el no

el erro que me arroja es este: Uncaught TypeError: Cannot set property 'visibility' of undefined

por lo que entiendo de ingles me dice que la propiedad visibility esta uindefinida

pero en los campos que quiero sacar la tengo definida como visible y por defecto chekeado el radio que dice "si".

[PHPeros]

Hm...Hay varios elementos con nombre "galeria", por eso genera un array. (Pasa en los dos)

document.formNoticias.galeria[index].style.visibility="hidden";

Lo que debería ir en index es el índice del array según su posición en el DOM empezando de 0.
Otra opción mucho más cómoda es usar ID.

[WorseMzg]

Iniciado por PHPeros

Hm...Hay varios elementos con nombre "galeria", por eso genera un array. (Pasa en los dos)



Lo que debería ir en index es el índice del array según su posición en el DOM empezando de 0.
Otra opción mucho más cómoda es usar ID.

nop tampoco me resulta, de echo intente solo dejar un input con ese nombre y nada no desaparece se corrigieron todos los errores de sintaxis posible