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

Responsive grid layout – Testimonials

Daniel 160

@DanoSvK

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

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

@KrzysztofLeczycki

Posted

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
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