Base Apperel Using Simple HTML, CSS & JS.

Community feedback

• 

  Amon•2,560

  @A-amonPosted over 3 years ago

  Hello! Great job~

  I have a few suggestions for you (that could level it up! 😀)

  • Put the main content of your site inside main tag. (Then, regenerate the report)

  • Put input element, .error and #errorImg inside a div. This way, when you absolute position your #errorImg, it will be relative to the div instead of the form. (So that you don't have to manually set the value for #errorImg's left to 17rem. Instead, you just need to set right:0)

  • The alt describes the image but in the case of your submit button, it describes the action. E.g. Send email/ Submit.

  • Instead of setting the style directly in JS to show/hide something, add/remove or toggle class instead. E.g. adding .error class to show error-related styles.

  I learnt about validity.valid, thanks to you! 😀

  Marked as helpful

  1

  ---

  

  P

  Skyz Walker•1,045

  @Skyz03Posted over 3 years ago

  @A-amon Thanks for the valuable feedback will surely implement this out.

  0

  

  Amon•2,560

  @A-amonPosted over 3 years ago

  @Skyz03 Glad to help!

  0