Responsive solution using Flexbox and Media Queries
Design comparison
Community feedback
- @rachit0706Posted about 2 years ago
Congratulations for completing the challenge!
I do have a suggestion for you to improve your code. Your body element is not taking the full height of the viewport and the component also is not at the center of the page as depicted in the design. For that you can use the following CSS in your code.
body { min-height: 100vh; display: grid; place-items: center; }
This will ensure the body takes the proper height and component will be at center.
Also instead of using <h3> tag for heading you should use <h1> as it is suggested in the accessibility report . You can then give it a proper font-size.
If you find my feedback valuable then please mark this as helpful. Cheers!
Marked as helpful1
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