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

Mobile first testimonials section with grid

@joaskr

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


Hi! I will be thankful for any comments, advice and feedbacks :-)

Community feedback

T
Grace 29,310

@grace-snow

Posted

This looks really nice code Joanna.

On your grid, remember you can use shorthand for css values like gap: 25px 25px; - no need to have two values if your row and col gap are the same 😊

And on your testimonial-header, you could make that a header html element if you wanted. Not necessary, but would make sense.

The only thing I question in your html markup is the h2s. They don't feel like they should be headings to me. If you imagine the whole document content written out in plain text, what would be the heading? I expect it would be the person's name....

But really these are down to personal choice I think.

Good job

1

@maxenceyrowah

Posted

I find your work super impressive Joanna Skrzypczak but I have a little problem when I open the chrome console to test the responsive there is a small bug in the centering

0

@joaskr

Posted

@maxenceyrowah Thanks! I'll fix it :-)

0

@maxenceyrowah

Posted

@joaskr great 👍

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