Design comparison
SolutionDesign
Community feedback
- @VCaramesPosted about 2 years ago
Hey @BayoOtomuola, some suggestions to improve you code:
- To center you content to your page, add the following to your Body Element:
body { min-height: 100vh; display: grid; place-content: center; }
- This entire component should have been built using CSS Grid as the challenge name implies. If you are not comfortable using CSS Grid or do not know how use it, here is a link that is all about how to use CSS Grid:
https://css-tricks.com/snippets/css/complete-guide-grid/
-
The “30-day, hassle-free money back guarantee” is not a heading. It should instead be wrapped in a Paragraph Element.
-
"Join Our Community" is an <h1> heading and all the other headings are <h2> headings.
-
Button hover color is incorrect You want to look at the “style-guide” to find the correct color.
-
The
background
color for the “Why Us” section is hsl(179, 47%, 52%);
Happy Coding! 👻🎃
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