Office Lite Landing Page

Community feedback

• 

  Davide•1,725

  @Da-vi-dePosted over 3 years ago

  Hi, nice result on this challenge, although it's not mobile first approach!

  • There's a typo in your HTML, the closing main tag misses /, also i'd recommend checking the report and try to resolve the issues.

  • In JS you add the class show but in CSS that class has display: block it should be display: unset because you set display: none to email__error, email__success but this isn't enough yet. First you need to remove the class in your else statement, you can't just add the class, you also need to remove it! And second either use an id for selecting emailSuccess and emailError or different class, styled classes shouldn't be used for selectig.

  Hope it helps, happy coding :-)

  Marked as helpful

  1

  ---

  

  Nick•760

  @Nick331102Posted over 3 years ago

  @Da-vi-de Thank you for the feedback

  1