Blog Preview Card

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 10 months ago

  Hello @Masood-zone!

  Your solution looks great!

  I have a suggestion for improvement:

  • For semantic reasons, use <main> to wrap the main content instead of a <div>.

  📌 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
• 

  Marcos Travaglini•4,840

  @BlackpachamamePosted 10 months ago

  Greetings! you have done a great job 😎

  📌 Some accessibility and semantics recommendations for your HTML

  • To improve the semantics of your HTML, you can change your <div class="container"> to a <main class="container">
  • You don't need margin: auto to center the content. You can add the justify-content: center property to your body to achieve the same result
  • Texts must always go in a header or paragraph tag. Think about which label makes the most sense for each thing. Greg Hooper text should be inside a p or a header. The span tag is only used to give a different style to a part of a text

  0