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 Testimonials-grid-section

Albaβ€’ 70

@albscr

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


Hey there! Here is a new challenge, I hope you like it. Looking forward to your recommendations. Thanks!

Community feedback

Lucas πŸ‘Ύβ€’ 104,580

@correlucas

Posted

πŸ‘ΎHello again Alba =) , Congratulations on completing this challenge!

I liked a lots the customization you've done here with the card gradients. Amazing.

You’re in the right track I can see that you’ve used the majority semantic tags possible for this challenge, the only block you’ve missed is the paragraph containing the quote text you can improve the accessibility there using <blockquote> to indicate to screen readers that the content inside that paragraph is a quote.

✌️ I hope this helps you and happy coding!

0
AntoineCβ€’ 1,180

@AntoineC-dev

Posted

Hi Alba. Great job completing this challenge πŸŽ‰πŸŽ‰πŸŽ‰.

It look really great and minus some small accessibility issues stated in your report that you can easily fix; I have only two small advices to give you.

I would increase the font-size of the .text paragraphs in your cards and increase the opacity a bit because it it really small right now which makes it hard to read. I am not that old 😁 and even I have some difficulties reading them.

I would also make sure that the card-image content always stay vertically centered by adding the property align-items: center on the container.

That's all! Again great job and happy coding 😊 Peace.

0
Adrianoβ€’ 33,960

@AdrianoEscarabote

Posted

Hi Alba, how are you?

I really liked the result of your project, but I have some tips that I think you will like:

1- Page should contain a level-one heading click here

2- All page content should be contained by landmarks, you can understand better by clicking here: click here

I noticed that the content is stretching a lot in higher resolutions, to fix this we can do the following:

I added:

.container {
    max-width: 1110px;
}

The rest is great!!

Hope it helps...πŸ‘

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