Cita:
Iniciado por maycolalvarez 
dentro de un bloque javascript no hace falta colocar javascript:
te faltan algunos ; y verifica errores de sintaxis con una consola tipo firebug
yo que tu hubiese utilizado un ciclo y obtener todos los inputs con getElementsByTagName
No lo entiendo, no sé mucho de js pero es que ese código lo usé con un form de 2 input y me funcionó de la siguiente manera :
Código PHP:
function comprobar(){
if ((document.getElementById("correo").value.length==0) || (document.getElementById("password").value.length==0 )){
javascript:cartelmax.className='max';
document.getElementById("correo").focus()
return 0;
}else{
document.iniciar.submit();
}
}
Lo que pasa es que como el form de ahora tiene 4 inputs, no sé como hacerlo de esa misma manera, alguna ayuda?