Product preview card component

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi Bitu,

  Congratulations 🎉 for completing another challenge, your Card component looks great, and it is great that you used flex to center the card. I have some suggestions/issues I want to address:

  1. It is important to use the correct semantics which I have mentioned in your previous challenge.
  2. To make your button look more like a button, you can use the cursor property e.g. cursor: pointer;
  3. According to the design, the price '$149.99' is quite large to make the price stick out and that it is on a discount.
  4. I noticed that you took a desktop-first approach however I would strongly suggest taking a mobile-first approach as it will be easier for responsiveness and changing/rearranging any elements within your container, as you increase the screen size.

  Overall, great work and wish you the best for your future projects so keep coding 👍.

  Marked as helpful

  0