Sunnyside-agency

Community feedback

• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  hey good job on this challenge! your report above shows a few issues though:

  • you should have at least one <h1> in your webpage

  • add alt="" to your <img> tags

  here is a list i made of accessibility issues & best practices

  hope this helps :))

  Marked as helpful

  0
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Awesome job with this one. Looking at your solution, I would suggest the following for you...

  • Remove the <section class="header"> in the header tag
  • Instead of using nav for the footer-menu and social-links, utilized unordered or ordered list like ul or ol and most common is div or span
  • Place description to the alt in the img , don't leave it blank
  • Clean the Whitespaces and space in the code
  • Add cursor: pointer for the hover state of the hamburger menu

  Overall you did well and Keep it going!

  0