Design comparison
SolutionDesign
Solution retrospective
Since the 3 headings have the same level of importance, I wasn't sure what level to use. I made them all h2, but should I make one an h1?
Community feedback
- @HassiaiPosted almost 2 years ago
You should have made the headings h1 instead of h2 , you have an accessibility issue because of that.
There is no need to give the body a width and height value. Add min-height to the main to center .options-card using flexbox
Use rem for as the unit for the max-width and the width, for more on css unit watch https://youtu.be/N5wpD9Ov_To
Hope am helpful Happy coding
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