Falla en Validacion de Buscador

[Gustavo1973]

Tengo este formulario

Código HTML:

<table height="10"cellpadding="0" cellspacing="10"border="0">
<tr>
    <td height="10"valign="middle">
    <form method="POST"action="?id=buscador">
    <a class="subtitulo">Buscador: </a></td><td valign="middle"><input id="busqueda"type="text"name="busqueda"size="15"></td>
    <td valign="middle"><input type="submit"value="Buscar"onClick="validab(this.form)">
    </td>
    </form>
</tr>
</table> 

El Script que lo Valida es el siguiente

Código:

<script>
function validab(form)
{
if (form.busqueda.value == "")
{ 
	alert("Debe ingresar la Busqueda"); 
	form.busqueda.focus(); 
	return ;
}
form.submit();
}
</script>

Cuando ejecuto la busqueda sin escribir nada en el textbox, me salta el Alert, pero igual se ejecuta la busqueda, este script lo use muchas veces en formularios de contacto y funciona bien, alguien ve algo mal??
Saludos y gracias

[zerokilled]

la validacion tienes que hacerla en el evento onsubmit para que puedas cancelar el envio en caso de ser necesario. aunque lo anterior es la opcion preferible tambien puedes cambiar el tipo de boton de submit a button y con eso sera suficiente.

[caricatos]

Hola:

Como estás usando un botón del tipo submit, ese return debe devolver false, y por la misma razón sobra el submit forzado... y la validación es algo que debe hacerse desde el evento submit del formulario y no desde el evento click del botón...

<form onsubmit="return validab(this)" ...

Saludos

[Gustavo1973]

UFFF
Eso me pasa por toquetear el codigo.
Cambie submit por button y andubo joya, gracias
Saludos

[caricatos]

Iniciado por Gustavo1973

UFFF
Eso me pasa por toquetear el codigo.
Cambie submit por button y andubo joya, gracias
Saludos

Si te sirve, tal vez esté bien para tí, pero se trata de un formulario "inaccesible". Siempre es mejor los botones del tipo submit para enviar el formulario, y por eso de la accesibilidad una doble validación (otra en el servidor )

Saludos

[quiquedt]

mete el form.submit(); en el else y no te pasará eso.


saludos

[caricatos]

Iniciado por quiquedt

mete el form.submit(); en el else y no te pasará eso.


saludos

Hola:

Justamente lo de form.submit() es lo que hace que sea inaccesible

Saludos

[quiquedt]

Lo de la accesibilidad tienes razón, pero me he leído algunas veces el post y no veo por ningún lado que quiera hacerlo accesible...
Por lo demás pues...te puedes ahorrar la patada en la boca...se supone que estamos para ayudar.

[caricatos]

Iniciado por quiquedt

Lo de la accesibilidad tienes razón, pero me he leído algunas veces el post y no veo por ningún lado que quiera hacerlo accesible... se supone que estamos para ayudar.

Justamente una forma de ayudar es dando buenos consejos, y si con un par de arreglos se consigue la accesibilidad, porqué no comentarlo.

Iniciado por quiquedt

...te puedes ahorrar la patada en la boca......

Sólo se trata de un emoticón... no le dés vueltas a un simple "click".

Saludos

[Gustavo1973]

A que se refieren con inaccesible???
Saludos

[zerokilled]

lo que quiere decir es la dependencia de una aplicacion a ciertos recursos. en tu caso, la aplicacion solo funciona si tiene javascript disponible. pero si se da el caso de un navegador que no tiene soporte para javascript, o bien el usuario lo ha inhabilitado, tu aplicacion deja de funcionar. por tanto, se vuelve inaccesible y excluye a cierta parte de la poblacion. lo correcto seria realizar primero la aplicacion que sea funcional sin recursos externos y luego sobre esa base incorporar los demas recursos. encontre este sitio que demuestra ejemplos de web inaccesibles.

[Gustavo1973]

Muchas Gracias!!!
Es que mis conocimientos de javascript aun estan muy bajos
Gracias y Saludos

[caricatos]

Hola:

Para que veas que con pocas modificaciones puedes hacer tu código accesible:

Código:

<table height="10"cellpadding="0" cellspacing="10"border="0">
<tr>
    <td height="10" valign="middle">
    <form method="POST" action="?id=buscador" onsubmit="return validab(this)">
    <a class="subtitulo">Buscador: </a></td><td valign="middle"><input id="busqueda"type="text"name="busqueda"size="15"></td>
    <td valign="middle"><input type="submit" value="Buscar" ><!--onClick="validab(this.form)"-->
    </td>
    </form>
</tr>
</table>

En azul lo añadido y en rojo lo quitado...

Y la validación:

Código:

<script>
function validab(form)
{
if (form.busqueda.value == "")
{ 
	alert("Debe ingresar la Busqueda"); 
	form.busqueda.focus(); 
	return false;
}
//form.submit();
}
</script>

Saludos

[Gustavo1973]

Gracias Caricatos, lo voy cambiar por tu codigo.
Saludos y Gracias nuevamente