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

Abu Salmanโ€ข 190

@AbuSalman143

Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Mohammed Abbasโ€ข 390

@Mohammedabbas7

Posted

Hello

Congrats on completing this challenge๐ŸŽ‰๐ŸŽ‰๐ŸŽ‰๐ŸŽ‰.

Nice solution and also like what you did with the attribution.

I noticed a few things in your solution you need to adjust:

  • First instead of adding height: 100vh to the body element because this causes some overflow issue so add min-height: 100vh to make the responsive.
  • Second instead of adding height: 60% to the .content element just add gap property gap: 1rem.
  • you did add a height property to .container element that is not good practice, so don't add specific height to elements unless it's so important to add it.

I hope my advice is useful.

Happy Coding :)

0

Abu Salmanโ€ข 190

@AbuSalman143

Posted

@Mohammedabbas7 Thank you so much for the valuable advice given. In the future, I will make sure to apply your advice to further projects.

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