Design comparison
SolutionDesign
Solution retrospective
Any tips and tricks are welcome.
Community feedback
- @maciejkrol18Posted about 2 years ago
I would suggest improving the
hr
tag's appearance. As of now, it's very bold and doesn't really match the provided design.You can style the
hr
tag with theborder
property. The line will look way better with something likeborder: 1px solid #e3e3e3;
applied. You can also try to use other color models such as hsl instead of hex code.Frontend Mentor's accessibility report mentions that all content should be contained by landmarks. You can fix that by changing the
div
with the class ofcontainer
that contains the card to amain
tag. Look into Semantic HTML.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