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

Testimonials Grid Section

P

@stefan-holmgren

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?

Enjoyed the way one can utilize "grid-area".

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

Mostly trying to keep the html light, and not as css-dependent as I usually do it.

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

Not really sure.

Community feedback

P
cloudpc7 180

@cloudpc7

Posted

Your design looks picture perfect, but I notice that when you are using padding you are only giving 3 inputs, when doing padding I believe that it takes 4 top left down and right. also if the body is already declared without margin or padding @ 0, why put the body: unset again. And why unset just put margin: 0 padding 0; but this should go into the css reset stylesheet and * {} which is already in the css stylesheet.

0

P

@stefan-holmgren

Posted

@cloudpc7

Using only three values is valid. If you're using four values it's in the order Top, Right, Bottom, Left.

If you're using three you define Top, Right/Left, Bottom.

Also the unset stuff is part of the CSS reset I'm using - a copy/paste thingy that I haven't looked that much into I'm afraid 😟

Using two would be Top/Bottom, Right/Left

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