Skip to content
Submitted over 3 years ago

Stats preview card components finished challenge

Ivan Holguin•170
@Ascecent
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Stats preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello everyone.

This is my first challenge in this platform, I've tried my best so hope you enjoy it.

Any feedback will be pleasantly received :D.

Community feedback

  • Riyana Gueco•495
    @rnguecoPosted over 3 years ago

    Hi Ivan!

    Great job on this! 👍

    There are a few minor details with regards to semantic HTML: heading level elements (h1, h2, h3, etc.) are usually used for topics and subtopics, so it does not make sense to use it for the statistics for this particular content since 10k+ or 314 or 1M does not really denote a topic or a subtopic. Also, it would be better if you put your statistics in a list, it makes semantic sense to do so.

    But overall this was a fantastic first try! Cheers and happy coding! 🎉

    Marked as helpful
    1
    Ivan Holguin•170
    @AscecentPosted over 3 years ago

    @rngueco Thank you so much for the feedback, i've updated the project with your suggestions :D

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord