A solution for the Product Preview Card Component Challenge

Community feedback

• 

  Mykola Klievanskyi•30

  @klievanskyi

  Posted almost 3 years ago

  Hey @Salah1221!

  Good job! Looks great and is very similar to the design.

  I have a couple of suggestions to improve your solution.

  1. It's not semantic to use ul and li tags to wrap the product card. Better to replace it with div.
  2. It's better to use button tag because a user will not redirect to another page after clicking. The central role of such cards is to add products to a bucket.
  3. You could remove the optical view version of the font. It's the main reason why the title is not the same as the design on the screenshot.

  I hope my notice will help you.

  11

  Marked as helpful