Product-Preview-Card---Component-1st-Project

Community feedback

• 

  Patrick•800

  @PPechmannPosted over 2 years ago

  Hey Avatar,

  very well done!

  I can relate to your comment, as I used to struggle with parts of code and solved them without knowing exactly how I solved them.

  I am also still learning, but I highly recommend taking your time to understand why exactly the code does what it does (Google is your best friend). It might take more time to finish a project initially but in the long term it pays off to take a moment and understand the why behind a solution.

  Congrats again on the challenge, you nailed it!

  Patrick

  Marked as helpful

  0
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Avatar, congratulations for your solution!

  I've checked your live site and is working fully responsive, what was your doubt?

  To improve it a bit you can use more semantic tags, for example <main> to wrap the component and <picture> to wrap both desktop and mobile images. Note that using picture you can set the image changing between devices without need to set the media query, you can set these changes entirely in the html.

  Hope it helps and happy coding!

  Marked as helpful

  0