stats-preview-card-component

Community feedback

• 

  Ramy Ibrahim•105

  @RamyEbrahim

  Posted about 4 years ago

  well done :)

  01
• 

  Agata Liberska•4,075

  @AgataLiberska

  Posted about 4 years ago

  Hi @rabahmilano! Well done on this challenge, it looks great! I only have two (really minor) comments:

  • on very small mobile screens, the image doesn't fully cover the container. Not a massive issue, but you could easily fix that by changing background-size to cover.

  • I think the stats part could be a ul here - but again, not a massive issue, but it's good to use semantic HTML where it makes sense :)

  Again, nice work!

  03