Social-links-profile-main(HTML and CSS)

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted 7 months ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have a suggestion regarding your code that I believe will be of great interest to you.

  ---

  HTML 🏷️:

  • Your solution may cause accessibility errors due to lack of semantic markup, which causes lacking of landmark for a webpage and allows accessibility issues to screen readers.

  ---

  • Landmarks are used to define major sections of your page instead of relying on generic elements like <div> or <span>. They are use to provide a more precise detail of the structure of our webpage to the browser or screen readers

  ---

  • So resolve the issue by replacing the <div class="contenedor"> element with the proper semantic element <main> in your index.html file to improve accessibility and organization of your page

  ---

  • We converted the <div class="contenedor"> into<main class="contenedor"> because it includes all content directly related to the page's main idea.

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  0