Huddle landing page v3 solution

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Hey, great work on this one. The layout in desktop is good and that loading animation is good, a bit of animation touch right. The mobile layout is good as well.

  Some suggestions would be:

  1. Using hierarchical header tags. You started at h2 on the main title, where it should have been h1. Following order from h1-h6 is good. Also you used h1 on the two quantity title which should not be the case, since we should only use 1 h1 per page.

  2. It is not a good practice to remove the outline on the elements. Those outlines serves as a visual key to know where a user is currently in. If you were to remove it, make sure to add some :focus state on those, either box-shadow or anything that will state where they are .

  3. Social media links should ne wrapped inside their respective a tags. Since they are links right, it is much better for them to be nested in a tags.

  Just those, but still, you did a good job here^

  0

  ---

  

  Harsh Jain•475

  @thisisharshjainPosted over 3 years ago

  Hey @pikamart, Thanks a lot for checking out my solution and pointing out the mistakes i did. I fixed them and updated the solution page ✌ Have a great day!

  0