Huddle-landing-page

Community feedback

• 

  Muhammad Shajjar•1,100

  @muhammadshajjarPosted about 3 years ago

  Hello, congrats on completing your FEM challenge, yep you approach the design as needed but at some viewports, your content is overflowing and some other suggestions that would help you with your design and solution

  • Background-image is repeating, use background-repeat:no-repeat;
  • Add max-width to a body or make a container class that wraps your all content, by adding it you can control your content from growing too much on larger viewports and save yourself from this situation
  • The logo should be in anchor a, logos are used to navigate on the home page in most the cases
  • Your images should have alt attribute but for decorative images, you should leave alt as blank alt="" and use role=" presentation" or aria-hidden=" true" to make sure all screen readers ignore those images. In your case, the illustration image is decorative.
  • To overcome your accessibility issue you would need to add some semantic markers to designate sections of the page as the header, navigation, main content, and footer e.g:


  <main>
  <div class”wrapper”> 
  </div>
  </main>
  

  Apart form this very well done, keep up the great work

  Marked as helpful

  1