Submitted almost 3 years ago

Testimonials grid section using SCSS, CSS Grid and Flexbox

Moises• 60

@devmoises

Preview site View code

This is a solution for...

Testimonials grid section

HTML
CSS

2junior

View challenge

Design comparison

SolutionDesign

Solution retrospective

Practicing CSS Grid, I really like this property and I put my knowledge into practice, any contribution and help is welcome. Thanks.

Community feedback

Mohit Menghnani• 55

@mmenghnani

Posted almost 3 years ago

This looks really great. I have a minor things that I was thinking upon

All img tags seem to have the photo class. Can you just use the img tag for the style?
For the alt text, should it just say Daniel Clifford or something like "image of Daniel Clifford". I am curious on this one myself

But I love this work, congrats and all the best. Cheers!

Marked as helpful

Moises• 60

@devmoises

Posted almost 3 years ago

Hello @mmenghnani

1.- Yes, it would work perfectly adding the styles to the img tag, in fact I removed the class. 👍

2.- Of course, for accessibility issues if it would be better to complement the alt attribute with more information, for now I will leave it with Photo of Daniel Clifford ✌

Thanks for the input.

Patrick• 14,325

@palgramming

Posted almost 3 years ago

Looks really good but you are missing the " image in the Daniel card

Marked as helpful

Moises• 60

@devmoises

Posted almost 3 years ago

Hi @palgramming, if I forgot, thanks for the detail, I just updated it. Greetings.

Please log in to post a comment

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