Design comparison
SolutionDesign
Community feedback
- @frankgomezdevPosted 9 months ago
My advice and feedback would be:
- Be a little more descriptive with your class names.
- Try using an <a> tag for the links seeing as they can be clicked and would direct you to a different page.
- The color change to yellow should be on the ":hover" state of the links instead of active. (I might be wrong about this, gotta go double check the design).
- Take a look into using Flexbox or CSS Grid. Once you understand it, it's way easier to create this layout without having to use fixed heights and widths and pay attention to the padding and spacing between elements.
Great job and keep up the good work!
Marked as helpful1
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