huddle landing page using flexbox

Community feedback

• 

  P

  AjeaS•715

  @AjeaSmithPosted over 4 years ago

  Awesome job! like @emmy-html said, your solution is very close to the design. So again, awesome job on that. I noticed that you have a few accessibility issues, which I see are an easy fix. It's basically saying for best practices when working with links <a></a> tags, if the link is not actually going anywhere like to navigate to a different page it can look like this <a>my link</a>. Or, if you are navigating to a different page then, using the href attribute like you did before is the way to go. Nonetheless, great work!

  P.S. hope that made sense :)

  1

  ---

  

  Ornella T•425

  @ornel77Posted over 4 years ago

  @AjeaSmith oh ok yeah that make sense thank you so much :)

  1

  

  P

  AjeaS•715

  @AjeaSmithPosted over 4 years ago

  @ornel77 your very welcome!

  0
• 

  emmy-html•345

  @emmy-htmlPosted over 4 years ago

  Hi! I think it looks great! The responsiveness is really good and it doesn't look like you missed anything when it comes to the design vs your solution. One suggestion I have is about the smallest size available. In the footer, all of the content is left aligned. I think if the "Huddle" logo and the Facebook, Instagram and Twitter content was centered, it would look better aesthetically. Great job!

  1

  ---

  

  Ornella T•425

  @ornel77Posted over 4 years ago

  @emmy-html ah ok thank you very much :)

  1

  

  emmy-html•345

  @emmy-htmlPosted over 4 years ago

  @ornel77 no problem!

  0