Skip to content
Submitted over 3 years ago

Testimonials grid section

Sabine•260
@LadyoftheCave
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

I first tried to make this exercise in flexbox. But that didn't work for me. Since I didn't know CSS grid, I watched a youtube video and after that video I made this exercise. As a result, I am assuming that there are areas of improvement. Feedback is appreciated.

Community feedback

  • Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    Great work even though like you said, was it your first grid styling? Well apart from that, tips. As I can see in your mobile state or breakpoints rather, you added grid-row to the child elements right. Well it works but it will be better if your container have a grid-template-rows property, just to you know, keeping it semantically styled. You could added like repeat(5, auto) something like that. But hey, good work even though its a first^

    0
    Sabine•260
    @LadyoftheCavePosted over 3 years ago

    Hi Pikamart. Thank you for your reply, I will work on my code.

    0
    Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    @LadyoftheCave Don't forget to commit so we could see the update^

    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