Skip to content
Submitted over 3 years ago

Responsive site, solution, GRID , FLEXBOX.

Antony William•50
@AntonyWilliam
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

So, that was my solution, any inputs?

Community feedback

  • Андрій Рогов•1,155
    @Andrii-RohovPosted over 3 years ago

    Hi, nice solution. But you should try to remove width from body tag. And remove from container div (position: absolute; top: 20%; left: 15%;) and try to add (margin: 150px auto;) and see the result

    Marked as helpful
    1
    Antony William•50
    @AntonyWilliamPosted over 3 years ago

    @Andrii-Rohov i did what you said, and now it looks better! thanks!

    0
    Андрій Рогов•1,155
    @Andrii-RohovPosted over 3 years ago

    @AntonyWilliam 👍

    0
  • Victor Barbosa Rocha•110
    @VictorrrochaPosted over 3 years ago

    I know a good practice is to zero the padding in the <body>

    You could use flexbox on the <body> to center your container div, instead of setting an absolute value.

    Marked as helpful
    1
  • guthribm•130
    @guthribmPosted over 3 years ago

    Well done! 👍👏

    1

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