Results summary component

Solution retrospective
Any form of feedback is highly appreciated. Thank you in advance
Please log in to post a comment
Log in with GitHubCommunity feedback
- @SoulRvr29
You can center your
.container
in a better way. Add tobody
:body{ min-height: 100vh; display: grid; place-content: center; }
or if you want to use flexbox:
body { min-height: 100vh; display: flex; flex-direction: column; align-items: center; justify-content: center; }
Then you can remove in
.container
margin: 95px 400px
andposition: fixed
. Also remove the properties in.attribute
, you don't need them.Avoid setting elements to a fixed height. The width is better to be set with
max-width
. Good luck, and happy coding!Marked as helpful
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