Fix issue OpenClassrooms-Student-Center#1 close modal and issue OpenC…

…lassrooms-Student-Center#2 required elements
metalfeanor · Apr 3, 2021 · cabd35a · cabd35a
1 parent 74e7ea1
commit cabd35a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/starterOnly/index.html b/starterOnly/index.html
@@ -71,6 +71,7 @@ <h1 class="hero-headline">
                   id="first"
                   name="first"
                   minlength="2"
+                  required
                 /><br>
               </div>
               <div
@@ -81,6 +82,7 @@ <h1 class="hero-headline">
                   type="text"
                   id="last"
                   name="last"
+                  required
                 /><br>
               </div>
               <div
@@ -91,6 +93,7 @@ <h1 class="hero-headline">
                   type="email"
                   id="email"
                   name="email"
+                  required
                 /><br>
               </div>
               <div
@@ -101,6 +104,7 @@ <h1 class="hero-headline">
                   type="date"
                   id="birthdate"
                   name="birthdate"
+                  required
                 /><br>
               </div>
               <div

diff --git a/starterOnly/modal.js b/starterOnly/modal.js
@@ -10,14 +10,19 @@ function editNav() {
 // DOM Elements
 const modalbg = document.querySelector(".bground");
 const modalBtn = document.querySelectorAll(".modal-btn");
+const modalCloseBtn = document.querySelectorAll(".close");
 const formData = document.querySelectorAll(".formData");
 
 // launch modal event
 modalBtn.forEach((btn) => btn.addEventListener("click", launchModal));
+// close modal event
+modalCloseBtn.forEach((e) => e.addEventListener("click", closeModal));
 
 // launch modal form
 function launchModal() {
   modalbg.style.display = "block";
 }
-
-
+// close modal form
+function closeModal() {
+  modalbg.style.display = "none";
+}