Huddle landing page with alternating feature blocks using Flexbox
Design comparison
Solution retrospective
Feedbacks Highly are recommended
Community feedback
- @FluffyKasPosted over 2 years ago
Hey there,
Your solution works pretty well on all screen sizes! Apart form the things mentioned already above, I'd add a few small things:
-
You should only have one h1 heading for your website, usually for the main title. For the rest, you go for lower level headings, decreasing them by one. This video explains this very well and it's an important thing to keep an eye on.
-
For your footer links and social media links you may wanna switch
div
toul
as they're lists. -
Aria labels: instead of saying "facebook link", just say "Facebook". An
a
element stands for a link so there's no need to specify it again in the aria-label what it is. -
I'll leave an article here about alt texts, you might find this useful!
Apart from these smaller semantic issues i think you did a really good job on this challenge! Well done and happy coding! ^^
0@Alone-07Posted over 2 years ago@FluffyKas Thanks for your feedback. I will correct them all you mentioned.
Again Thank you!!
0 -
- Account deleted
Hi there 👋
Congratulate on finishing your project 🎉. You did a great job 🔨
I give some suggestions that I hope help you take your project to the next level 📈.
- I think the buttons look smaller. Let's make them bigger
- I think the logo is not showing in the footer. When I opened it didn't work on my browser at least 👍💡
Happy coding ☕
Maqsud
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