Submitted over 3 years ago

Responsive site, solution, GRID , FLEXBOX.

Antony William• 50

@AntonyWilliam

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Stats preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

So, that was my solution, any inputs?

Community feedback

Андрій Рогов• 1,155

@Andrii-Rohov

Posted 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

Antony William• 50

@AntonyWilliam

Posted over 3 years ago

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

Андрій Рогов• 1,155

@Andrii-Rohov

Posted over 3 years ago

@AntonyWilliam 👍

Victor Barbosa Rocha• 110

@Victorrrocha

Posted 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

guthribm• 130

@guthribm

Posted over 3 years ago

Well done! 👍👏

Please log in to post a comment

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