Skip to content
Submitted over 3 years ago

Stats Preview (SCSS, BEM, Semantic Html, Mobile & Desktop Responsive)

Franco A.•200
@franco-a
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! This is my first submission on the site! I'd like to know if my use of semantic HTML and BEM was done correctly. If there are some best practices that I have missed then I would very much appreciate it if someone could point it out.

Community feedback

  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted over 3 years ago

    Greetings, Franco A.! 👋

    Congratulations on completing your first Frontend Mentor challenge! 🎉 Good job on this one! 👏 Your solution looks good, is responsive, and your HTML and BEM look good to me! 👍

    This is a very minor thing but I think the card could be a little wider in the mobile layout as long as there's room for it to grow. You might want to consider adding something like width: 100% and then a max-width (so that it doesn't become too wide at certain screen sizes) to the card to do just that. 😉

    Keep coding (and happy coding, too)! 😁

    1
    Franco A.•200
    @franco-aPosted over 3 years ago

    @ApplePieGiraffe I'll keep that in mind. Thank you for your feedback!

    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