Skip to content
Submitted over 2 years ago

Responsive grid layout – Testimonials

Daniel•160
@DanoSvK
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Testimonials grid section
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

I have been learning Grid layout for like 5 days. What do you think about my solution? Also, it tends to overflow and setting width and height is very tricky. How do you solve that? When I am trying to just make it higher or wider, flexbox stops working correctly.

Community feedback

  • Krzysztof•240
    @KrzysztofLeczyckiPosted over 2 years ago

    Hi, The fIrst of all don't duplicate id atributes in your tags. Each id shall be unique. This code snippet: body {width: 90rem;} spoils your layout. Just remove it. By the way I think using percentage or view units for setting dimentions of the block elements is more comfortable than rem. You could use minmax function, auto-fit or more media queries to improve responsivness of your grid. Best wishes. :D

    2

Please log in to post a comment

Log in with GitHub

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