Huddle landing page

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, great work on this one. The layout in desktop looks good and the mobile state as well. But the responsiveness is not really. Right now the layout does not scale with screen changes, you might want to look at that one.

  Some suggestions would be:

  • Lose the word logo on the website image. Avoid adding those words that relates to "graphic" such as "logo, image, icon". Assistive tech will handle those for you.
  • Do not nest the footer inside the main element, have it in its own row in the html. A typical structure would look like:

  <header /> # if there is a header
  <main />
  <footer />
  

  • The register should have used a tag and not button since it is more like a go to link to sign up instead of signup pop-up modal.
  • Lose the word illustration on the hero image and maybe more clearer alt value like alt="huddle social media mockup"
  • Those social media links should use a tag since those are links.
  • Each a tag that wraps the social media links should have aria-label attribute on it, the value would correspond to the social media it is bounded to. For example, the a tag that wraps the facebook should have aria-label="facebook". This way, users will know where this link would take them.

  Aside from those above, great work. But the responsiveness should be improved. If you finalizes a layout, make sure to inspect the website in dev tools, change the screen sizes, check if layout looks good.

  Marked as helpful

  0

  ---

  

  Carlos Eduardo•160

  @carlosecosmesilvaPosted about 3 years ago

  Thanks for tips! @pikamart

  0