Skip to content
Submitted over 4 years ago

Testimonial Grid using CSS Grid & Flexbox

Chris-Ann C.•300
@cdc2021
A solution to the Testimonials grid section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hiya,

Any feeback would be welcomed specifically class spedification and responsiveness.

Thank you :)

Code

Please log in to post a comment

Log in

Community feedback

  • Gerben Dol•3,115
    @GerbenDol

    Hey Chris-Ann! Great job on the layout!

    There's a couple of things I'd keep in mind regarding the design:

    • The font-size is a bit small in my opinion, it's a bit hard to read
    • Make sure to check sizes and spacing within the cards. For example right now the profile photos are a bit too big and the margin below the photo is a bit small.

    It's details like this that will get your solution from looking good to looking great!

  • JK•345
    @JakubKepak

    Hi Chris-Ann C. ! Nice work 👏

    To centre all content to the centre of a page a would set display:flex to the body and then add justify-content: centre and align-items: centre. In order to work, you would also need to set the property height on the body element. For instance height:100vh. Also, I would maybe set max-width proepty to .wrapper in order not to fill all space in wide screens.

    Anyway, nice job! Keep improving 🙌

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