Design comparison
SolutionDesign
Solution retrospective
hey will love feedback from you guys
Community feedback
- @FluffyKasPosted about 3 years ago
Hey, good start on this challenge! There's a few things to improve.
- There's actually no need to add an extra container div. Everything you added there could be just applied to the body!
- Your button uses the browser's default sans serif. This always happens when working with buttons, unless you set the font-family specifically when you style the buttons. Alternatively, you can use
all: unset
to get rid of all browser defaults. As the accessibility report says, it's a good idea to swap the outer div tomain
to make your html more semantic.
Good luck! :)
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