Skip to content
Submitted over 3 years ago

Testimonials Grid Section using Sass, Grid, and Flex

Lance Martija•185
@lancemartija
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Testimonials grid section
  • HTML
  • CSS
2junior

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

  • Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    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
    Lance Martija•185
    @lancemartijaPosted over 3 years ago

    @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
  • Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    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
    Lance Martija•185
    @lancemartijaPosted over 3 years ago

    @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
    @AzkanorouziPosted over 3 years ago

    good job mate

    1
    Lance Martija•185
    @lancemartijaPosted over 3 years ago

    @Azkanorouzi Thank you, Azka!

    0

Please log in to post a comment

Log in with GitHub

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