Blog Preview Card - HTML, Tailwind CSS, Alpine

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 10 months ago

  Hello @Theosaurus-Rex!

  Your solution looks excellent!

  I have a couple of suggestions:

  • First: Use main to wrap the main content instead of main and article.

  📌 The tag article would make more sense if the card was part of a bigger website (in certainly would in real world), but here it is all we have on the screen.

  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.

  • Second: Since the title is a clickable element, it's nice to add cursor: pointer and a color shift as hover effect.

  📌 This gives the user a visual indication that the element is clickable, as obvious as it may look.

  I hope it helps!

  Other than that, great job!

  Marked as helpful

  1

  ---

  

  Theo Harris•140

  @Theosaurus-RexPosted 10 months ago

  @danielmrz-dev Hey Daniel, thanks so much for your feedback! That makes a lot of sense regarding the article tag - I guess I was approaching it like I would approach building a reusable component for a larger page, but it makes sense to treat it differently in this case! I'll apply your suggestions and update my solution 😄

  1

  

  Daniel 🛸•44,250

  @danielmrz-devPosted 10 months ago

  @Theosaurus-Rex

  I'm glad I could help! 😊

  1