buongiorno...

ho un problema alquanto "stupido" ma non capisco come mai mi salta la lettura di una funzione che ho creato...

compilo i dati di un form premo il submit per inviarli, e invece di controllare se i dati sono compilati o meno, lo script salta questo controllo e invia ugualmente i dati... come mai?

il codice è il seguente

funzione controlla

codice:
<script>
Modernizr.load({
    test: Modernizr.mq('only all'),
    nope: 'js/vendor/respond.min.js'
});
$(document).ready(function(){
$('a[href*=#]').click(function() {
    if (location.pathname.replace(/^\//,'') == this.pathname.replace(/^\//,'') && location.hostname == this.hostname) {
        var $target = $(this.hash);
        $target = $target.length && $target || $('[name=' + this.hash.slice(1) +']');
        if ($target.length) {
            var targetOffset = $target.offset().top;  
            $('html,body').animate({scrollTop: targetOffset}, 1000); 
            return false;
        }
    }
});
$('.contattami a img').css('border','0px');
});
function controlla(){
    if((document.getElementById(mail)=="") || (document.getElementById(subj)=="") || (document.getElementById(messaggio)=="")){
        alert("prova");
return false;
    }
}
</script>
ho incollato tutto lo <script> magari sbaglio qualcosa nell'inserimento che non mi accorto...

il form invece è questo

codice:
<form method="post" onsubmit="return controlla()" action="sendmail.php">
<input type="text" name="email" id="mail" placeholder="Inserisci inidirizzo e-mail" />
<input type="text" name="subj" id="subj" placeholder="Inserisci un oggetto" />
<textarea name="messaggio" id="messaggio" placeholder="Inserisci testo del messaggio"></textarea>
diciamo che il comando onsubmit viene saltato proprio.. passa subito all'action