Newsletter sign-up form with success message solution

Community feedback

• 

  Tushar Biswas•4,080

  @itushPosted over 1 year ago

  🎉 Congratulations on completing the challenge! Great job! 👏

  Your solution looks nice. However, I'd like to point out a few suggestions to make it even better. 🤓

  • It looks like you forgot to close the </body> tag in your HTML.
  • ⚡️ While it's alright to link a JavaScript file in the head element with the defer attribute, it's best practice to link JavaScript files at the end of the body. This ensures that HTML parsing is complete before JavaScript execution. If you need to execute JavaScript code earlier, defer is a good option.
  • You may safely remove the email validation script from the body below the footer and it should work.

  Once again, you have done a wonderful job! 🌟

  I look forward to seeing more cool projects from you in the future🚀

  Keep up the great work and happy hacking! 💻

  1

  ---

  

  Sonsawan Ngamsom•140

  @murphy6867Posted over 1 year ago

  @itush I'm so appreciative of your comment. You reminded me of a lot of things that I had overlooked 😂. Thank you for cheering me up, and I will try my best to improve my front-end skills. 🙏🏻

  1
• 

  Paks•1,350

  @LifeofPaksPosted over 1 year ago

  Look really good. 👍

  1

  ---

  

  Sonsawan Ngamsom•140

  @murphy6867Posted over 1 year ago

  @LifeofPaks Thanks bro. 🤙

  0