Submitted 12 months ago
I have used grid layouts and flex to center the card container
@mujtabaghafari
Design comparison
SolutionDesign
Solution retrospective
3-column preview card component #html #css #js
Community feedback
- @danielmrz-devPosted 12 months ago
Hello, @MujtabaGhafari!
Your project looks very good!
Here's one suggestion for you to improve it:
- I noticed that your
.card-container
is growing and taking 100% of the screen width. If you set a lowermax-width
, it'll prevent that from happening and your project will look a lot closer to the original design. I tested reducing themax-width
to 900px and it had a very good effect:
max-width: 900px; <<<<<<< Here width: 100%; display: grid; grid-template-columns: repeat(3, 1fr); border-radius: 10px; box-shadow: 5px 5px 10px 5px rgba(0,0,0,0.15);
I hope it helps!
It's not like you really need that, your project already looks great 😊. It's just a suggestion, in case you wanna improve it.
Marked as helpful0@mujtabaghafariPosted 12 months ago@danielmrz-dev, thank you so much for the feedback and I really, really appreciate that!
1 - I noticed that your
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