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

Testimonials-Grid-Section-Main(Solution)

Johan BB 300

@JohanXTheKing

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Community feedback

@kkalvaa

Posted

I like how you're using grid for this and that you're changing the amount of columns based on viewport width. However, on some widths you're letting the columns get a bit small which leaves the 1 column cards with a very narrow text. You should look into Una Kravets RAM grid: https://codepen.io/una/pen/oNbvNQv

The cards really shouldn't be figures, they should be divs or article. <figure> is meant for images and stuff, but this content is text based.

Marked as helpful

0

Johan BB 300

@JohanXTheKing

Posted

@kkalvaa Thanks for the advice, I have already made the corresponding modifications. :)

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