huddle-landing-page-with-alternating-feature-blocks-master

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  Regarding you question,

  There is nothing wrong with the way you did it using Flex. As for the media queries that is fine too.

  If you wish to learn more about responsiveness, here is a link to Google Developer’s site that will teach you how make it 100% responsive:

  https://web.dev/learn/design/

  • The Alt Tag description in the logo needs to be improved upon. The description should state the company name.

  • The Illustrations serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  • The Article element is not the best choice for wrapping each section. In order to use the Article element the component needs to be able to make sense on its own and be independently distributable (can be used in on another site).

  • The footer “phone” and “address” should be wrapped in Anchor Tags, so users can easily click on them and have the appropriate app open for them.

  • The footer links need to be wrapped inside a nav element and should only be one single list.

  • Your "social media icons" are not displaying properly. You will want to check your code.

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding! 🍂🦃

  Marked as helpful

  0