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

I have used grid layouts and flex to center the card container

@mujtabaghafari

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


3-column preview card component #html #css #js

Community feedback

Daniel 🛸 44,230

@danielmrz-dev

Posted

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 lower max-width, it'll prevent that from happening and your project will look a lot closer to the original design. I tested reducing the max-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 helpful

0

@mujtabaghafari

Posted

@danielmrz-dev, thank you so much for the feedback and I really, really appreciate that!

1
Daniel 🛸 44,230

@danielmrz-dev

Posted

Glad I could help 😊 @MujtabaGhafari

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