Fylo landing page with two column layout

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 1 year ago

  Hello there 👋. Good job on completing the challenge !

  I have some suggestions about your code that might interest you.

  • In your design, the links in the footer and header should be wrapped inside an unordered list (<ul>) and then contained within a <nav> element. This provides a clear semantic structure and helps screen readers understand the navigation.

  • The aria-hidden="true" attribute is used to hide an element from accessibility tools like screen readers. However, using it alongside an alt attribute can create confusion, as alt provides accessibility information while aria-hidden tries to hide the element. This combination may lead to contradictory behavior for assistive technologies.

    In the case of logos and images within div.user for testimonials, you generally don't want to use aria-hidden="true" because these are typically meaningful content that should be accessible to all users, including those using screen readers.

  I hope you find it useful! 😄

  Happy coding!

  Marked as helpful

  1

  ---

  

  Amens11•190

  @Amens11Posted about 1 year ago

  Hi @MelvinAguilar, thanks for your useful suggestion. I will update my code based on your comments.

  1