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