Responsive landing page using grid/flex

Community feedback

• 

  Maximilian Dybvik•200

  @RegexRiddlerPosted 6 months ago

  Good job on completing the challenge 😁🎉

  I have a suggestion and a recommendation for you!

  • When a user has typed a valid email and the success "page" is displayed, try reflecting the user input email in the body text instead of the statically typed one.
  • And when a user inputs its own email there is really no reason to show the user input as an a tag with an href since the user is not going to email themselves.

  <p class="popup-description"> A confirmation email has been sent to 
    <a type="email" href="mailto:[email protected]">[email protected]. 
    </a> Please open it and click the button inside to confirm your subscription. 
  </p>
  

  Could be something like this:

  // index.js
  const formInput = document.querySelector('.form-input');
  const userEmail = document.querySelector('#user-email');
  
  function validateForm() {
    // If input passes validation, set userEmail to the value of formInput
    userEmail.innerHTML  = formInput.value;
    ...
  }
  
  // index.html
  <p class="popup-description"> A confirmation email has been sent to 
    <span id="user-email"></span>. Please open it and click the button inside to 
    confirm your subscription. 
  </p>
  

  Here is my submission, and my GitHub repo if you feel like comparing code.

  0
• 

  P

  luccyyy97•140

  @luccyyy97Posted 6 months ago

  nice 1

  0