Skip to content
Submitted 5 months ago

Responsive layout using CSS Grid and Media Queries

Mohammed Majd•120
@MajdMohammed
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?

I am proud I could manage to use the CSS grid.

What challenges did you encounter, and how did you overcome them?

I encountered challenge with the scaling of the body padding, I used clamp to solve the problem.

What specific areas of your project would you like help with?

When to use certain untis like em. How can I calculate the dimensions in the design so I can clone it better.

Community feedback

  • Kezzer•110
    @KezzerDFPosted 5 months ago

    Does the solution include semantic HTML? I think so Is it accessible, and what improvements could be made? Class names Does the layout look good on a range of screen sizes? Its good Is the code well-structured, readable, and reusable? Yup Does the solution differ considerably from the design? Very close

    1
    Mohammed Majd•120
    @MajdMohammedPosted 5 months ago

    @KezzerDF Thank you for your feedback on those areas of project.

    Although all the positive things in my code, I found many mistakes, Like i forget the SVG file, then I use only one blockquote tag instead of 2, I changed the dimension of the layouts, I edited the grid columns and rows, etc...

    You're welcome to re-check if you're interested ^^

    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