Testimonials grid section

Community feedback

• 

  Adriano•38,120

  @AdrianoEscarabotePosted 18 days ago

  Hello Pickle, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

  Using Flexbox or Grid on the body to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed with margin, padding, or absolute positioning. These techniques provide more consistent alignment and simplify the code.

  flexbox:

  body {
      display: flex;
      justify-content: center;
      align-items:  center;
      min-height: 100vh;
  }
  

  grid:

  body {
      display: grid;
      place-content: center;
      min-height: 100vh;
  }
  

  The rest is spot on.

  Hope it’s helpful to you. 👍

  Marked as helpful

  0

  ---

  

  Pickle•220

  @Augurk66Posted 16 days ago

  @AdrianoEscarabote

  Hello Adriano! I am fine and thank you for your feedback. I really appreciate this tip, because I am still struggling to find a correct way to center all the elements in the body. I used this technique before, but I keep changing it. Maybe it is good idea to standardize my methods.

  Have a good day!

  1
• 

  1lija•140

  @1lijaPosted 19 days ago

  Regarding your issue with horizontal gap between name and verified, you could've made a div that contains both of them, then you could play around with either display flex, maybe even use a grid there or just use gap. You should've centered the container but that's a quick fix. Other than that, all looks good

  Marked as helpful

  0

  ---

  

  Pickle•220

  @Augurk66Posted 16 days ago

  @1lija

  Will try that, thank you!

  0