Product preview card component

Community feedback

• 

  Prantik•660

  @prantiknoor

  Posted about 3 years ago

  Hey Mohamed Metwally, Congratulations! 🎉🎉 You have completed you first challenge in Frontend Mentor.

  I want to give a recommendation to use picture to switch between images.

  <picture class="image">
      <source media="(min-width: 850px)" srcset="images/image-product-desktop.jpg">
      <img src="images/image-product-mobile.jpg" alt="product image" />
  </picture>
  

  11

  Marked as helpful
• 

  kirolos•0

  @kiroxd

  Posted about 3 years ago

  Hi Mohamed, your design is great but you can add a line break after "Gabrielle" and a line break after"Eau" to get it looking as close to the design as possible, and you need to add space between every line in the paragraph and you need to some margin at the top of your container

  hope I helped you

  11
• 

  Divine Obeten•2,415

  @Deevyn9

  Posted about 3 years ago

  Hi Mohamed, great representation, your card container would need some margin at the top for mobile, and the margin between the paragraph and pricing needs to be reduced.

  Happy coding 🎈

  12