
Design comparison
SolutionDesign
Community feedback
- P@drewleePosted about 5 hours ago
The page is functional and overall fairly close to the spec. I'd suggest a few improvements to more accurately match the visual design.
- The black heading color is harsh and should be set to the "Slate 900" variant that is established in the style guide.
- Same issue for the section markers (e.g., "01", "02"), which should be the "Slate 600" variant.
- The main heading should be a smaller font size for mobile screens.
- The user pictures in the grid should feature a border-radius.
- The footer background color should be "Cyan 600" instead of black.
Additionally, I would look into leveraging more semantic HTML elements for your markup, such as
heading
,main
,footer
,section
, andarticle
elements.Div
s should just basically be reserved for stylistic only purposes or used as a last resort when there are no better options.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