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

@mohammedreddah

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


What challenges did you encounter, and how did you overcome them?

Hello everyone!!

thankfully i didn't face any problem.

any suggestions are welcomed .

What specific areas of your project would you like help with?

Hello everyone!!

thankfully i didn't face any problem.

any suggestions are welcomed .

Community feedback

Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello there!

Congrats on completing the challenge! āœ…

Your solution is really impressive!

I've got a couple of ideas (about how to use HTML better) that could make it even stronger:

šŸ“Œ First: Think about using <main> to wrap your main content instead of <div>.

Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

šŸ“Œ Second: Don't skip heading levels ā€“ start with <h1>, then use <h2>, and so on.

It's more than just text size ā€” it's about structuring your content effectively:

  • The <h1> to <h6> tags are used to define HTML headings.
  • <h1> is for the most important heading.
  • <h6> is for the least important heading.
  • Stick to just one <h1> per page ā€“ it should be the main title for the whole page.

These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

Hope that's helpful!

Keep up the great work!

Marked as helpful

1
Dylan de Bruijnā€¢ 3,190

@DylandeBruijn

Posted

@mohammedreddah

Hiya! šŸ‘‹

Congratulations on your solution, it looks very close to the design! I can tell you put a lot of effort into it.

Things I like about your solution šŸŽ‰

  • Clear descriptive CSS classes

Things you could improve āœļø

  • Add a bit of padding to your body element so the card has room to breath on smaller viewports.

  • Try changing the height: 100vh on your body to min-height: 100vh so the body is 100vh at a minimum but still being able to grow in height when the content grows.

  • Try using CSS variables to make your values more reusable across your code.

  • Try experimenting more with Flexbox.

  • Try using relative CSS units like rem and em.

  • Try putting an a tag in your title with a href attribute so you get the default click functionality.

  • Try making your solution responsive.

I hope you find my feedback valuable, and I would appreciate it greatly if you could mark my comment as helpful if it was! šŸŒŸ

Let me know if you have more questions and I'll do my best to answer them. šŸ™‹ā€ā™‚ļø

Happy coding! šŸ˜Ž

Marked as helpful

1
HazelChambokoā€¢ 40

@HazelChamboko

Posted

Thats a good one right there

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