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

P

@galonaranjo

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?

  • Tightening up my workflow.
  • Flexbox is starting to feel comfortable.
  • I want to improve my layout skills.

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

I still have quite a bit of looking things up on some basic syntax, but that will come with time.

Community feedback

@sherlineau

Posted

Hey there! You have some redundant css styles that can be cut out. You only display flex for one area, which is the author info div. Setting the container class and div to flex with a flex-direction of column is the same as just leaving the default display to block. So you could condense lines 11-21 to just being .authorInfo {display: flex}. You can adjust the margins for your meta and tag classes to achieve the "spacing" needed.

Marked as helpful

0

P

@galonaranjo

Posted

@sherlineau Great points Sherline. Thanks for the feedback, I'm going to make the changes now.

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