Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted over 3 years ago
It looks on that preview like you might be missing the background color on the page, and a max-width on your component?
0@grace-snowPosted over 3 years agoA few other pointers for you ☺
- if a button in a design would trigger navigation to a page, it should be using the anchor tag, not button tag
- always use a responsive unit like rem for font sizes, never pixels. It ensures proper font scaling for those who need to do that to see a site
- why limit the width of container on desktop? I'm not sure you need to do that as that seems to be acting as the page background. I'd just have a max width on the cards if that is the case
- if you define something in one min width media query, you don't need to repeat it in the next one up
- don't forget about transitions, and giving interactive elements visible focus states.
I hope these help you. Happy coding ☺
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