cambierei
EmailAddr = f.mail.value;
Filtro = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
if (Filtro.test(EmailAddr))
return true;
else
{
alert("Controlla l'indirizzo di e-mail inserito");
f.mail.focus();
return false;
}
in
EmailAddr = f.mail.value;
Filtro = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
if (!Filtro.test(EmailAddr))
{
alert("Controlla l'indirizzo di e-mail inserito");
f.mail.focus();
return false;
}
inoltre toglierei il "return true" alla fine della funzione