Responsive insure Landing page using NEXTJS, Styled-components, TS

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Heyo,

  Your solution looks great! Responsiveness seems fine as well. I only noticed some smaller things you could perhaps double-check:

  1. In mobile view, there's no gap between the first and second row of footer links. The titles of the second row is touching the first row's last links.

  2. It would be nice if you wrapped your links (footer links, nav links and social icons as well) in <a> tags, so they're actual links.

  3. Your hamburger menu button should be a <button> with aria-label and aria-expanded attributes added.

  4. The images with the features section could use an aria-hidden.

  Happy coding ^^

  Marked as helpful

  0