Huddle landing page with HTML and CSS

Solution retrospective
I tried to make this responsive beyond the original design! Any suggestions are welcome
Please log in to post a comment
Log in with GitHubCommunity feedback
- @FluffyKas
Heyo,
Awesome job on this one. Looks very good on all screen sizes (apart from the extra purple background colour which hurts my eyes a bit, if I'm honest :D). I'd like to add a few things where you could improve this a bit:
-
Wrapping buttons in an
a
tag is incorrect. You either go with a link or a button, you can't do both :) They do slightly different things, in this case, just using a button will do! -
The div for your social links could be swapped for a
ul
, it's considered a list. Your links also need to have an aria-label, as they don't have any visible text-description. -
For your logo alt text, you don't need to specify it's a logo (it's an image element so it's sort of self-explanatory), "Huddle." would be a better alt for it.
Apart from the above mentioned things you did really well! I remember it was kinda tricky to make it look good on all screen sizes when I was doing this challenge so well done. :)
Marked as helpful -
Join 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