Blog Preview Card

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Hello @ekahanny!

  Your solution looks great!

  I have a couple of suggestions for improvement:

  • For semantic reasons, and since that is the main title of the screen, you can replace the <h3> with <h1>. This change is not just about the size of the title.

  The <h1> to <h6> tags are used to define HTML headings. <h1> defines the most important heading. <h6> defines the least important heading. Only use one <h1> per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>, then use <h2>, and so on.

  • Also, you can add a hover effect to the title, to indicate that's a clickable element. Just a color shift and cursor indicator is enough:

  .title:hover {
    color: yellow;
    cursor: pointer;
  }
  

  I hope it helps!

  Other than those details, your solution is great!

  Marked as helpful

  0

  ---

  

  ekahanny•70

  @ekahannyPosted 11 months ago

  @danielmrz-dev thank you for your feedback

  0