Submitted over 3 years ago
Huddle landing page using html and css.
@Muchiri400
Design comparison
SolutionDesign
Solution retrospective
This is my first challenge to submit. It's not picture perfect and would like some constructive feedback. Thank you.
Community feedback
- @tedikoPosted over 3 years ago
Hello, Antony Njenga Muchiri! 👋
Congrats on finishing your first challenge! 🎉 Your solution looks very good and also responds well. Here's my few tips:
- Add
background-size: cover;
to your body to to fill the container with background. - Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
- Instead of adding
margin-left: 20px
to your anchor elements with icons addFlexbox
gap
property to.social-links
container. For example:gap: 20px
. With gap spacing, we only add space between the items. Take a look in dev tools when you add margin-left value that EVERY item has that property, so it will causes problems in diffrent scenarios when your first or last item has margin-left/right set. Gap is easy way to remove that problem.
Good luck with that, have fun coding! 💪
1@Muchiri400Posted over 3 years agoThank you tediko. I'll definitely look into that.
0 - Add
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