Css Flexbox

Solution retrospective
looking forward for your feedback and advice.
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@palgramming
- you are currently missing your border radius on your layout
- you currently have cards wrapping into two rows when the browser windows is about 1140px wide. So you need to work on your transitions point between desktop and mobile
Marked as helpful - @vanzasetia
👋Hi Emad Jaber! My name is Vanza!
I have some feedbacks to help you improve this solution:
- The
background-color
should be very light gray, since that's how it is on the design. - The content should be on the center of the page, to achieve that you can try this:
body { display: flex; align-items: center; justify-content: center; min-height: 100vh; }
flex-wrap: wrap
on.container
cause an issue when only 2 cards that can fit on a row, try to fix that by adding more media queries.
That's it! Hopefully this is helpful!
Marked as helpful - The
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