Gdzie jest bląd? JAVASCRIPT

pytanie zadane 29 stycznia 2019 w JavaScript przez kuba1245677 Obywatel (1,080 p.)

Nie wiem czemu ten kod nie dziala

<!DOCTYPE HTML>
<hmtl>
<head>
	<meta charset="UTF-8">
	<title>Przyklad</title>
</head>
<body>
	<form method="post">

		<input type="checkbox" id="terms" />
		<div id="termsHint"></div>

		<input type="submit" value="Zarejestruj sie" id="formSignup"/>

	</form>

	<script type="text/javascript">
			
		var elForm, elTerms, elTermsHint;

		elForm = document.getElementById("formSignup");
		elTerms = document.getElementById("terms");
		elTermsHint = document.getElementById("termsHint");

		

		function checkTerms(event){
			if(!elTerms.checked){
				elTermsHint.innerHTML = "Musisz zgodzic sie z warunkiem!";

				event.preventDefault();
			}
		}

		elForm.addEventListener("submit",checkTerms, false);
		

	</script>

</body>
</hmtl>

1 odpowiedź

odpowiedź 30 stycznia 2019 przez Riddick Bywalec (2,600 p.)
wybrane 30 stycznia 2019 przez kuba1245677

Najlepsza

id="formSignup" jest na przycisku wysyłającym a powinno być na formularzu czyli na tagu form. Przycisk nie jest wysyłany tylko cały formularz.

komentarz 30 stycznia 2019 przez pablop76 VIP (123,120 p.)

Sugerował bym raczej skorzystać ze zdarzenia click na przycisku oraz przenieść

preventDefault()

przed warunek

if(!elTerms.checked)

komentarz 30 stycznia 2019 przez Riddick Bywalec (2,600 p.)

Nie zgadzam się, preventDefault przed warunkiem uniemożliwi całkowicie wysłanie formularza a zdarzenie click nie służy do wysyłania formularzy. Pomyśl co się stanie jeśli podczas wypełniania formularza użytkownik kliknie enter na innym inputcie a nie kliknie myszką w przycisk.