Responsive page using CSS Grid

Solution retrospective
General feedback, please
Please log in to post a comment
Log in with GitHubCommunity feedback
- @yasssuz
Hello Yolanda,
Awesome build, but I would change some things:
- on the element container inside your
style.css
you wrotealign-items: center;
andjustify-content: center;
but you have adisplay: grid
. That's incorrect becausealign-items: center;
andjustify-content: center;
are adisplay: flex;
property, and not a grid one. - inside your container, I would put a
max-width: 1150px
so it will not expand through the entire screen. but then it will not be centered, so add amargin: auto
as well. - now your container is attached to the top, no problem, easy fix. Go to your body element and add
display: flex; align-items: center; justify-content: center; flex-direction: column; min-height: 100vh;
happy coding and if i was helpful please upvote my comment :)
- on the element container inside your
- @Karimsamir112
this is very good, I like it .
keep going and happy coding
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