Design comparison
SolutionDesign
Solution retrospective
Hi People this is my solution for this challenge thanks and happy coding
Community feedback
- @hitmorecodePosted about 1 year ago
Nice looks good, just a few tips
- Make it a habit of adding
min-height: 100vh;
on the body. This is important for responsiveness - If you remove margin on the
.container
and add flexbox on the body, you can place the card in the middle of the page. - If you prefer grid on the body add this (see below)
body{ font-size: var(--font-body); font-family: var(--font-family); background-color: var(--color-light-gray); min-height: 100vh; display: grid; place-items: center; }
- On the Why us section you used a
<p>
tag and adjusted the width of the text content. There are better ways to do this. You could have use a<p>
tag for each description or you could have used a<ul>
<ul> <li>text</li> <li>text</li> <li>text</li> </ul>
Marked as helpful1 - Make it a habit of adding
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