Design comparison
SolutionDesign
Community feedback
- @hectorlil48Posted 4 months ago
Hey, it's looking good. I would suggest that you wrap everything inside a div like you have but inside of the main tag. Having proper semantic HTML tags will help screen readers navigate through your page. Also taking a look at the style guide and taking the time to add all the colors to a :root tag would be helpful. That way you always know what colors to use from the design.
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