Skip to content
Submitted over 4 years ago

Testimonials grid created with CSS Grid and Flex {Begginer}

Enol Igareta•115
@Enol-Igareta
A solution to the Testimonials grid section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Good morning. I did this project using grid and flexbox. Any advice about my code will be very useful to continue improving, I would be very grateful if you could guide me or tell me any error you see in my code. Thank you very much for your time

Code

Please log in to post a comment

Log in

Community feedback

  • Szymon Rojek•4,540
    @SzymonRojek

    Hi Enol

    Great job! :D

    I have checked your HTML structure, a few tips from me:

    • if you want treat this challenge as a single page component then you can add h1 tag with the class sr hidden for the screen readers;
    • also you can remove style CSS from the code and transfer them to the CSS file;

    RWD:

    • add max-width to the container;
    • what do you think about tablets? you can go from one column (mobiles), then create two rows with two boxes each and the 5th box will be over the entire length of the container, finally will be the desktop design pattern. Let me know what do you think about it.

    That's it.

    Cheers :D

  • David Ouma•270
    @daudi13

    Hello, I've gone through your code and I've found it very well organised, responsive, clean and DRY. I've also learnt a lot from it. I did not really find any particular errors in your code. Keep it up. Happy coding.

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