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 using Sass, Grid, and Flex

@lancemartija

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 again! I would like to present to you my version of this project. I'm pretty sure I got it accurately, though what I'm absolutely sure of is that my Grid game is mediocre! Hahaha. Anyway, check it out and please don't hesitate to leave your opinion about my work.

I would love to know what you think I should improve with my HTML5 semantics and the way I use Grid so that I'll be able to improve them next time.

Thank you and keep on coding!

Community feedback

P
Chamu 12,970

@ChamuMutezva

Posted

So here is the issue:

  • your media query is kicking in very late - at min-width 1400px , most desktop are below that size and wont be able to view that grid implementation. The sizes for the designs are just a guide to get you going but the rest falls on you to produce a final product that can be viewed generally on all devices. You might need another media query that allows users on medium devices to be impressed, for example my desktop width is about 1380px - so viewing content that stretches that much does not inspire me

Good lucky

Marked as helpful

1

@lancemartija

Posted

@ChamuMutezva Yes, that's it. Thanks for the help! I'm currently taking a break, but I'll make sure to implement it right away.

1
P
Chamu 12,970

@ChamuMutezva

Posted

Greetings Lance

  • i am getting a different output, the cards are just stretching from left to right and does not reflect the design comparison shown above

Marked as helpful

0

@lancemartija

Posted

@ChamuMutezva I see. It must've been the breakpoints that I've implemented that didn't include the grid layouts for certain viewports. I'll make sure to fix it. Thank you!

0
Azka 480

@Azkanorouzi

Posted

good job mate

1

@lancemartija

Posted

@Azkanorouzi Thank you, Azka!

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