Вот ответ типа «Корона»: у меня было так много времени, что я сел и попытался переделать твой пример.
Мое первое впечатление: ты должен постараться не повторять себя (" DRY ": Не повторяйте себя). Это относится как к вашему HTML, так и к вашему JavaScript. Я удалил довольно много ваших идентификаторов, так как они сделают сценарии только сложнее и более подробными. Вместо этого я попытался проанализировать структуру вашего HTML. Сначала я просматриваю вашу форму и собираю все поля ввода, теги изображений и элементы div в массив с именем inps
. Это делается путем создания «правильного» массива путем применения [... <nodelist>]
к списку узлов, который я получаю от вызова document.querySelectorAll()
, отфильтровывая все
<input>
<img>
- и все элементы с
class="errorMessage"
Затем я применяю метод .reduce()
к этому массиву, чтобы создать еще один (inps
, массив объектов), в который
- я сначала помещаю объект
{inp:el}
, содержащий элемент <input>
, а после этого - я затем расширяю этот "последний" объект (
a[a.length-1]
) дополнительными свойствами (img
и div
), содержащий ошибку- <img>
и сообщение об ошибке- <div>
.
(Внимание: это будет работать правильно, только если является ассоциированным образом и сообщение об ошибке в поле HTML для каждого поля <input>
!)
Инициируется событием submit
. Затем я прохожу через этот тщательно подготовленный (inps
) массив, проверяю содержимое собранного поля ввода и выполняйте необходимые действия над связанными изображениями и делениями. Внутри вызова inps.forEach(e=>...)
я могу получить доступ к каждому элементу ввода с помощью e.inp
и связанных с ним изображений ошибок и div ошибок с помощью e.img
и e.div
.
Как упоминалось ранее, это был "раз Короны" введите упражнение "для меня, чтобы показать, что это возможно с относительно коротким кодом, но я настоятельно рекомендую использовать действительно хорошие HTML5 доступные методы (используя minlength
и соответствующие type
s для элементов ввода!).
const frm=document.forms[0];
const inps=[...frm.querySelectorAll('input,img,.errorMessage')].reduce((a,el,i)=>{
if (i%3==0) a.push({inp:el});
else a[a.length-1][el.tagName.toLocaleLowerCase()]=el;
return a;
},[])
// the following is superfluous, as it will always test as positive for <input type="email">:
const emailRX =/^[a-zA-Z0-9.!#$%&'*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*/;
frm.onsubmit=(ev)=>{
let err,allerrs=0
inps.forEach(e=>{
allerrs+=err=e.inp.value==''? 1 : e.inp.name=="email" ? ( emailRX.test(e.inp.value) ? 0 : 2 ) : 0;
e.img.style.display=['none','initial','initial'][err];
e.div.textContent=['',e.inp.placeholder+' must be filled!','Email appears to be invalid'][err];
})
return allerrs==0;
}
*,
*:focus,
*::before,
*::after {
margin: 0;
padding: 0;
outline: 0;
box-sizing: border-box;
}
body{
background-image:url(../images/bg-intro-desktop.png);
background-color: hsl(0, 100%, 74%);
font-family: 'Poppins', sans-serif;
height: 100%;
width: 100%;
}
.grid{
display: grid;
grid-template-columns: 33% 40%;
justify-content: space-evenly;
}
.h2{
color: white;
font-weight:bolder;
font-size: 3.2em;
}
.paragraph{
color: white;
font-size: 0.9em;
}
#firstH2{
margin-bottom: -14px;
}
.box{
background-color: white;
border-radius: 10px;
text-align: center;
height: auto;
box-shadow: 0px 9px rgba(0, 0, 0, 0.2);
}
.firstButton{
font-family: 'Poppins', sans-serif;
width: 100%;
border: 0;
color: white;
background-color: hsl(248, 32%, 49%);
font-size: 1.1em;
padding: 16px;
border-radius: 10px;
box-shadow: 0px 9px rgba(0, 0, 0, 0.2);
font-weight: bolder;
}
.firstButton span{
font-size: 0.8em;
font-weight: normal;
}
input {
font-family: 'Poppins', sans-serif;
width: 90%;
padding: 15px;
font-size: 1em;
border: 1px solid hsl(246, 25%, 77%);
border-radius: 5px;
font-weight: 600;
color: hsl(249, 10%, 26%);
}
input:focus{
font-family: 'Poppins', sans-serif;
font-size: 1em;
padding: 15px;
border: 1px solid black;
border-radius: 5px;
font-weight: 600;
color: hsl(249, 10%, 26%);
}
input::placeholder{
font-family: 'Poppins', sans-serif;
font-weight: 600;
}
.placeholder::placeholder{
color: hsl(0, 100%, 74%);
}
#greenButton{
width: 90%;
border: 0;
padding: 16px;
background-color: hsl(154, 59%, 51%);
border-radius: 10px;
box-shadow: 0px 5px hsl(154, 65%, 43%);
}
#greenButton span{
font-family: 'Poppins', sans-serif;
color: white;
font-size: 1.3em;
font-weight: bold;
}
#greenButton:hover{
cursor: pointer;
background-color: hsl(154, 94%, 65%);
}
.terms{
color: hsl(246, 25%, 77%);
font-size: 12px;
margin-top: 2%;
text-align: center;
}
.terms span a{
text-decoration: none;
color: hsl(0, 100%, 74%);
font-weight: bold;
}
form{
position: relative;
}
.errorMessage{
font-size: 0.8em;
color: hsl(0, 100%, 74%);
text-align: right;
font-style: italic;
font-weight: 500;
margin-right: 30px;
margin-top: -25px;
margin-bottom: 18px;
}
form img {display:none} /* hide all error-img first ... */
.attribution{
font-size: 11px;
text-align: center;
color: white;
}
.attribution a {
color: hsl(228, 45%, 44%);
}
@media only screen and (max-width:700px){
body{
background-image:url(../images/bg-intro-mobile.png);
}
.grid{
grid-template-columns: 90%;
}
.h2,.paragraph{
text-align: center;
}
}
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0"> <!-- displays site properly based on user's device -->
<link rel="icon" type="image/png" sizes="32x32" href="./images/favicon-32x32.png">
<link href="https://fonts.googleapis.com/css2?family=Poppins:ital,wght@0,400;0,500;0,600;0,700;1,300;1,400;1,500;1,600&display=swap" rel="stylesheet">
<link rel="stylesheet" href="css/style.css">
<title>Intro component With Sign up Form</title>
</head>
<body>
<div class="container">
<div class="grid">
<!-- 1st grid -->
<div>
<br><br><br><br><br><br><br><br>
<h2 class="h2" id="firstH2">Learn to code by</h2>
<h2 class="h2">watching others</h2><br>
<p class="paragraph">See how experienced solve problems in real time. Watching scripted tutuorials is great, but understanding how developers think is invaluable.</p>
</div>
<!-- 2nd grid -->
<div>
<div>
<br><br>
<button class="firstButton">Try it free 7 days <span>then $20/mo. thereafter</span></button>
<br><br>
<!-- box for white color -->
<div class="box">
<form>
<br><br>
<input type="text" name="firstName" placeholder="First Name" minlength="3"><br><br>
<img src="../images/icon-error.svg" alt="error">
<div class="errorMessage"></div>
<input type="text" name="lastName" placeholder="Last Name" minlength="3"><br><br>
<img src="../images/icon-error.svg" alt="error">
<div class="errorMessage"></div>
<input type="email" name="email" placeholder="Email Address"><br><br>
<img src="../images/icon-error.svg" alt="error">
<div class="errorMessage"></div>
<input type="password" name="password" placeholder="Password"><br><br>
<img src="../images/icon-error.svg" alt="error">
<div class="errorMessage"></div>
<button type="submit" id="greenButton"><span>CLAIM YOUR FREE TRIAL</span></button><br>
<div class="terms">
By clicking the button you are agreeing to our <span><a href="#">Terms and Services</a></span>
</div><br><br>
</form>
</div>
</div>
</div>
</div>
</div>
<br><br>
<footer>
<p class="attribution">
Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
Coded by <a href="#">Gurkiran Singh</a>.
</p>
</footer>
<script src="js/script.js"></script>
</body>
</html>