Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Blog Preview Card

ekahanny 70

@ekahanny

Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello! this is my submission for the Blog Preview Card challenge!

I'm open for any kind of feedback. Thank you.

Community feedback

Daniel 🛸 44,270

@danielmrz-dev

Posted

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

@ekahanny

Posted

@danielmrz-dev thank you for your feedback

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord