Responsive landing page using sass and bem

Hello, Felix! 👋

Congratulations on finishing this challenge! 👏 Nice work on this challenge! 🙌

I have some feedback on this solution:

• Accessibility
  • There's no header landmark in this challenge. The first section should be a section element that lives inside a main element. It's commonly known as hero section.
  • The logo images are important content of the website. They allow users to know on which website they are currently at, especially the users of assistive technology. Use the company name as the alternative text of the logo.
  • For the Download buttons, they should be link elements with the download attribute instead.
  • For any decorative images, each img tag should have empty alt="" and aria-hidden="true" attributes to make all web assistive technologies such as screen reader ignore those images. In this case, all images are decorative only, except the logo and social media icons.
  • Change all the h3 to p. The content below each heading is too small.
  • <a href="#" class="footer_social-link" title="facebook"> I would recommend using aria-label instead of title attribute. I don't think that title attribute would be pronounced by screen reader.
• Styling
  • I notice that you are still using px unit. I would recommend converting all the px to rem units. That way, everything will scale properly when the user changes the font size setting.

I hope this information is useful! Keep up the excellent work! 👍

Great job overall man. Keep it up