Design comparison
SolutionDesign
Community feedback
- @hitmorecodePosted 9 days ago
Congratulations looks good. I do have a few tips.
- Avoid using % unless it's necessary, otherwise just avoid it. It's true % make a page responsive, but it can also break a page when using it wrong.
- The cards are not centered on the page. To fix this add flexbox on the body and set the body height to 100vh.
- You need to do some changes to the media query to make it work on smaller screen sizes.
@media only screen and (max-width: 889px) { // remove whats commented out .container { flex-direction: column; /* display:table; */ /* margin: 10%; */ /* margin-right: 0%; */ }
I hope this was helpful. Keep it up 👌👍
0
Please log in to post a comment
Log in with GitHubJoin 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