Blog Preview Card

Community feedback

• 

  P

  Øystein Håberg•10,710

  @Islandstone89Posted 30 days ago

  Hey, excellent job! :)

  I only have a few small things to correct, so well done!

  HTML:

  • I would wrap the date in the <time> element:<p class="publish-date">Published <time datetime="2023-12-21">21 Dec 2023</time></p>.

  • I would change the heading to a <h2> - a page should only have one <h1>, reserved for the main heading. As this is a card heading, it would likely not be the main heading on a page with several components.

  • The link should be inside the heading, not the other way around.

  CSS:

  • Including a CSS Reset at the top is good practice.

  Marked as helpful

  0

  ---

  

  P

  Raphael Perfi•290

  @perfidevPosted 29 days ago

  Updated solution! Thanks again for your help.

  Sometimes I get confused about the semantics of the <a> tag.

  I didn’t know about the <time> tag; it helped me a lot. 🤯

  Lastly, I changed the <h1> tag to <h2>. I thought it wouldn't be an issue since it's a simple page, but it's always good to follow best practices.

  Thank you very much =)

  1