solution huddle landing page with alt feature blocks with flex wrap

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, great work on this one. Layout in desktop is good and it scales well when going in mobile layout. The mobile layout is good as well.

  Some suggestions would be:

  • The a tag that wraps the website logo should have aria-label="homepage". This way, a user would know where this link would take them, hence, homepage.
  • The try for free could be better with using a tag on it. Since it doesn't really look like a control element on this one, it is much more of a link.
  • I would not wrap the hero-section in the header I personally add the hero-section for the main element, because those acts like as the webpage's overview.
  • get started for free should be link as well a.
  • I think the hover effect for the interactive element's are bit too much in terms of size.

  FOOTER

  • The alt for the icons of the website information such as the location, telephone, email should have used alt="" on them, because it doesn't really add content.
  • The six links should have been using a tags and be inside a ul element, since those are list of links.
  • The a tag that wraps the social media link should have a aria-label which value corresponds to the social media name, like aria-label="facebook", this way users would know that this link would take them at facebook page.

  Other than those, great job.

  Marked as helpful

  0
• 

  P

  Kellen Ruyle•930

  @rule-kellsPosted about 3 years ago

  Thanks for the detailed feedback. I greatly appreciate it. I will keep these in mind for future projects.

  0