Skip to content
Submitted over 3 years ago

Flexbox and Grid

Bibek Pant•85
@pantbibek21
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

If you have time, please review the code. You will be highly appreciated.

Community feedback

  • darryncodes•6,370
    @darryncodesPosted over 3 years ago

    Awesome work Bibek. You responsive design is great.

    You may want to:

    • add background: hsl(233, 47%, 7%);' to the <body> not the .container
    • use semantic html. your .container <div> could be <main>. This should resolve the warnings in the accessibility report
    2
    Bibek Pant•85
    @pantbibek21Posted over 3 years ago

    @darryncodes Thank you ! Means a lot ! I will take care of that.

    0
  • Nuray•100
    @ganbnurayPosted over 3 years ago

    I think if you set flex:1; to .box the layout of .data would look much better.

    1
    Bibek Pant•85
    @pantbibek21Posted over 3 years ago

    @ganbnuray Thank you Nuray. What about rest of the code? Is there any thing I have to learn more?

    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