Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

CSS Grid Layout solution.

@sayyedaaman2

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

responsive design

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

grid system

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

typography

Community feedback

@ShoaibShuja

Posted

Amazing job my friend and the layout seems absolutely amazing but the page is overflowing a bit and even though you have centered the .wrapper inside the body it was done through setting margins but the better approach would be using flexbox and removing the margins to fix the overflowing issue.

Just in case these lines of code will help you center the .wrapper inside the body element.

body {
    width: 100%;
    min-width: 100vh;
    display: flex;
    flex-direction: column;
    justify-content: center;
    align-items: center;
}

And also, if you have an issue with putting the attribution (which says who coded the project) to the very bottom of the webpage use the position: absolute; property and then set the bottom: 0;.

I wish you best of lucks my friend. 👍

0

Please log in to post a comment

Log in with GitHub
Discord logo

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