Sunnyside Agency Landing Page

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Vishal Nirmal! 👋

  Good job on this one! 🎉 Your solution responds well and overall looks good. Here's my suggestions:

  • Change the alt attributes for the logo img, section backgrounds, .testimonial__item__image images, as they don't add any extra context for screen reader users. Since your images are decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies.
  • You should use anchor tags inside .menu__item li elements or in .section-2__elaborate-features__item__button so they behave like real links.

  Good luck with that, have fun coding! 💪

  Marked as helpful

  1
• 

  eruj22•410

  @eruj22Posted over 3 years ago

  Overall your design is quite good, but you can improve it a little bit. Possible improvements:

  • mobile menu should be wider
  • hamburger icon should have cursor: pointer
  • social icons on hover should turn white; following code can help you .footer__social img:hover { filter: invert(95%) sepia(5%) saturate(0%) hue-rotate(16deg) brightness(160%) contrast(107%); }

  Marked as helpful

  1