Product preview component Edugn2298

Community feedback

• 

  Luigi•240

  @luigi-peronePosted over 2 years ago

  Hi Eduardo, and welcome to the Frontend Mentor community!

  You've a great solution here but there are some tags you can consider changing in the html,the first div should be wrapped with a main tag and div with class attribution should be wrapped with a footer tag. These changes don't change the design but improve the accessibility and semantic.

  Hope it helps, happy coding 👋

  Marked as helpful

  0

  ---

  

  Eduardo Garcia•20

  @edugn2298Posted over 2 years ago

  @luigi-perone Hi Luigi. Thanks for the feedback. I'm a newbie and all the feedback is a lot of help.

  1