YK_Social_Media_Handles_Card

Community feedback

• 

  Mohamed Ahmed•120

  @fienmsaPosted 7 months ago

  Hello @codeyeskrish!

  Your solution looks great!

  I have a suggestion for improvement:

  Use <main> to wrap the main content instead of <div>. Tags like <div> and <span> are typical examples of non-semantic HTML elements. They serve only as content holders but give no indication as to what type of content they contain or what role that content plays on the page.

  📌 This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.

  I hope it helps!

  Other than that, great job!

  0