event.stopPropagation(); - этого не требуется.
if(this.name.value == "" || this.phone.value == ""){
value - это свойство, а не функция. Но в данном случае такая проверка некорректна так как значение " " пройдет такую проверку, а не является верным.
Лучше так, используя тот же JQ
if(!$.trim(this.name.value) || !$.trim(this.phone.value)) {
|