four card component using css grid

Please log in to post a comment
Log in with GitHubCommunity feedback
- @glrodriperez98
@Cipher126,
I checked out your Four Card Feature project and just wanted to say...great job overall! Your layout is super clean, your use of :root variables makes the CSS really organized, and everything feels polished and accurate to the design.
Here are a few things I really liked:
Grid Layout: Your use of CSS Grid to position the cards was awesome. I liked how you used specific grid-column and grid-row placements to arrange them uniquely — it mirrors the original design well!
Card Styling: Each card looks visually distinct with the top colored border using the ::before pseudo-element - something I have yet to dive into so nice job!
Responsiveness: I appreciated that you included media queries to make it mobile-friendly by switching to a column layout using Flexbox. That really helps the user experience on smaller screens.
A few small suggestions:
On mobile, the .card styles inside the media query have a grid-template-columns line that doesn’t apply to Flexbox — you might be able to clean that up.
You could consider using grid-template-areas to make your layout even more readable and maintainable, especially if you plan to iterate on the design more later.
But honestly, you're super close to pixel-perfect! You clearly put in a lot of effort...well done!
Kind regards, G
Marked as helpful
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