Skip to content
Submitted 8 months ago

Testimonials Grid Section

Stefan Holmgren•220
@stefan-holmgren
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?

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

  • cloudpc7•180
    @cloudpc7Posted 8 months ago

    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
    Stefan Holmgren•220
    @stefan-holmgrenPosted 8 months ago

    @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

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