Huddle landing page with alternating feature blocks using Flexbox

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 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:

  1. 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.

  2. For your footer links and social media links you may wanna switch div to ul as they're lists.

  3. 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.

  4. 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

  ---

  

  Krish•660

  @Alone-07Posted almost 3 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 📈.

  1. I think the buttons look smaller. Let's make them bigger
  2. 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