Base Apparel Landing page using CSS Grids and JS

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…

  • Wrap the whole content of container with specific tag like main and not <container> tag. For HTML structures, refer it with this one Semantics
  • Include description with the alt in image tags. Like this line <img src="./images/logo.svg" alt="" class="logo" />
  • Include media queries for the responsive design.

  Above all, Well done! Keep it up and Hope this is helpful!

  1
• 

  Shreyas Waykar•20

  @username-copiedPosted almost 3 years ago

  Thank You Rio :))

  0

  ---

  

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  @username-copied No worries! I would appreciate it if you find this as helpful! Cheers!

  1