Design comparison
Solution retrospective
I'm proud of using variables for spacing, colors and font size.
Community feedback
- @BuildAndBreakPosted 28 days ago
Hi, great job completing this project, @jeanethsn!
Your code is clean, readable, and makes effective use of semantic HTML.
I also appreciate the use of css variables, it's a great habit that will be invaluable in larger projects, so getting comfortable with them early is definitely a smart move.
A couple of suggestions:
-Fixed Width on Section:
I noticed you set a fixed width of 310px on the <section> element. While this will work on most devices, I’d recommend using a percentage (%) for more flexibility and adding a max-width of 310px if needed. This will make your layout more responsive across different screen sizes.
-Clickable Links:
Consider adding links within each list item to make the social media options clickable. Redirecting users to these platforms could be a nice enhancement and why not make your personal social links profile? Think about that :)
Tip: It looks like there are two closing <main> tags by mistake, so double-check that part.
Keep up the great work! Greetings, Tiago
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