Sunnyside agency landing page

Community feedback

• 

  P

  Dave•5,295

  @dwhensonPosted about 2 years ago

  Hi Isaque803

  Nice job here! Here's a few points you might like to consider:

  1. You could make the burger-menu element a button rather than a div. This will ensure that it can be opened by keyboard users.
  2. Perhaps have a look at the srcset attribute? This will means you won't need to always load two images to allow for wide/narrow screens, which is much better for performance.
  3. I think you need to put an accessible label on the svg icons at the bottom of the page? You could use aria-label (I think they should also be links?).

  Otherwise nice job on this one!

  Cheers Dave

  Marked as helpful

  1