Design comparison
Solution retrospective
- Tightening up my workflow.
- Flexbox is starting to feel comfortable.
- I want to improve my layout skills.
I still have quite a bit of looking things up on some basic syntax, but that will come with time.
Community feedback
- @sherlineauPosted 3 months ago
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 helpful0@galonaranjoPosted 3 months ago@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 GitHubJoin 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