Stats preview card component With CSS

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi Rian,

  Congratulations 🎉 for completing this challenge, your Stats Card component looks great, and it is great that you used flex to center the card. Also, it is great that you used the right semantic for your code. There are some issues/suggestions I want to address:

  1. Within your section tag that contains the pictures, there is a picture tag. I would suggest removing the section tag and just wrapping the pictures with a picture tag like you did as section tags usually contain a header, anywhere between h2 to h6.
  2. I noticed that you took a desktop-first approach however I would strongly suggest you take a mobile-first approach since it will be easier for responsiveness and it is best practice to do.

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

  Marked as helpful

  1