Submitted almost 4 years ago
Responsive four-cards using CSS with flexbox
@Vbanety
Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted almost 4 years ago
Hi Vinicius
This looks OK but I find it a bit of a shame you've not used css grid and have opted for transforms instead. It's just such a perfect opportunity for css grid practice, so maybe take another look.
Other small things to revisit:
- remember not to jump heading levels, they should go in sequential order
- try to match the design as closely as possible - your heading text is a lot darker and box shadows are noticeably different
That all said, you should be proud of this one. It's really cleanly written, easy to Understand and pretty close to the design overall. We'll done
1 - @VbanetyPosted almost 4 years ago
I got it about the sequential title, color title and box-shadow. I'll fix it later. Thanks for feedback.
Ps. I don't have too many practice with grid CSS, but I'm gonna to your advice and to study more about that.
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