Skip to content
Submitted about 1 month ago

Testimonialss-grid (flexbox + CSS grid)

Allan Sancley•110
@AllanSancley
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

What are you most proud of, and what would you do differently next time?
  • Me orgulho de ter concluido este desafio, e ter conseguido deixar a pagina mais proxima o possivel do design do projeto original.
What specific areas of your project would you like help with?
  • Gostaria de ajuda na parte de responsividade, sempre que coloco uma @media Queries a margem muda em outra @media Queries, como corrigir este problema?

Community feedback

  • Djamel1133•880
    @Djamel1133Posted about 1 month ago

    Hi,

    I liked your code and the way you organized it; it's very clean. However, I need to provide feedback as part of my learning path on Frontend Mentor. Here are my suggestions:

    1. I think using variables (for font-size, colors, etc.) will give you more flexibility. Check out this resource on CSS variables: W3Schools - CSS Variables.

    2. Use logical units (em, rem, etc.) instead of fixed values in px. There are some good articles on this topic here: Frontend Mentor - Building Responsive Layouts.

    Thank you and happy coding!

    1

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