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 | Flexbox

Shivianā€¢ 40

@Andre-DM

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 are you most proud of, and what would you do differently next time?

I didn't know how to import fonts from a local folder, I'm not sure if I did everything correctly, so I'd appreciate some feedback on that! I am not sure I want to continue using css nesting. In small projects it may be useful, but when facing more complex challenges I fear it may become too confusing later on. If you have any advice on this I welcome it!

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

I had difficulty rendering the image exactly as the design and... I didn't make it. Maybe I should have used background-image? And if so, how?

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

As I said above I had problems with the image, I also didn't understand how I was supposed to resize the font without using media queries.

Community feedback

Ryan Hardyā€¢ 110

@ryyHardy

Posted

I really like the way you wrote your CSS. I can tell you have a good foundation for this stuff. As for what you mentioned:

  • I feel like it's usually better to use Google Fonts, but this article has a pretty good summary of your options for loading fonts.
  • I don't mind the CSS nesting, but you can do the same thing using a descendant selector or something similar. Try looking at a reference of CSS selectors to see if there are alternatives to nesting because it isn't that widely used yet.
  • I think the image is good. All I noticed was that the padding on the card as a whole looks a little large.
  • As an alternative to media queries, I would try to mess around with the clamp() function. It's really good in some cases. I don't know for sure, though.

That's all I have to say. Everything else is really good.

Marked as helpful

1

Shivianā€¢ 40

@Andre-DM

Posted

@ryyHardy Hi there, firstly, thank you for taking the time to leave me feedback! I took a quick look at the article you linked and, as soon as I have time, I will definitely read it carefully since it seems very useful. Same with clamp(), I will definitely do some research on it. Thanks again for the tips!

0
Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello there!

Congrats on completing the challenge! āœ…

Your project is looking fantastic!

I'd like to suggest a way to make it even better:

  • Using margin isn't always the most effective method for centering an element.

Here's a highly efficient approach to position an element at the center of the page both vertically and horizontally:

šŸ“Œ Apply this CSS to the body (avoid using position or margins in order to work correctly):

body {
    min-height: 100vh;
    display: flex; 
    justify-content: center;
    align-items: center;
}

I hope you find this helpful!

Keep up the excellent work!

Marked as helpful

1

Shivianā€¢ 40

@Andre-DM

Posted

@danielmrz-dev Hi, thank you very much for the advice, as soon as I can I will modify the code with the one you proposed! If I may ask, why do you set min-height: 100vh?

0
neven.cebicā€¢ 510

@redizork

Posted

Look at my solution :)

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