huddle-landing-page-with-single-introductory-section

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 1 year ago

  Hi, Dipesh! 👋

  Congratulations on creating your first landing page at Frontend Mentor! 🎉

  Here are some suggestions for improvements.

  • Do not wrap everything with <main>: The <header> should be outside of the <main> element. Then, the <header> element should only wrap the logo. The rest of the content should not be wrapped by it.
  • Remove the word "image" from the alternative text: Any words that are related to "image" (for example, picture, photo, logo, icon, graphic, and avatar) should not be included in alternative text.
  • Decorative images should not have alternative text: Not every image needs alternative text. This will tell the screen reader to skip over the decorative images. As a result, it saves screen reader users time navigating the page. For your information, decorative images are images that do not add any information and serve only aesthetic purposes.
  • Social media links: All the social media icons should be wrapped by anchor tags.
  • Avoid id selectors: Do not use id selectors for styling. There are two reasons for not using ID’s to style content:
  • They mess up specificity because they are too high (the most important reason).
  • They are unique identifiers. So, they are not reusable on the same page.
  • Learn more — What the ID attribute is REALLY for
  • Modern CSS reset: Use a more modern CSS reset whenever you start a new project. My recommendation — A Modern CSS Reset | Andy Bell

  I hope this helps. Happy coding! 😄

  0