Testimonial Grid using CSS Grid & Flexbox

Community feedback

• 

  Gerben Dol•3,115

  @GerbenDolPosted almost 4 years ago

  Hey Chris-Ann! Great job on the layout!

  There's a couple of things I'd keep in mind regarding the design:

  • The font-size is a bit small in my opinion, it's a bit hard to read
  • Make sure to check sizes and spacing within the cards. For example right now the profile photos are a bit too big and the margin below the photo is a bit small.

  It's details like this that will get your solution from looking good to looking great!

  1

  ---

  

  Gerben Dol•3,115

  @GerbenDolPosted almost 4 years ago

  Oh, and I forgot to mention: your hover transition on each card is nice, but it snaps back with no animation when you leave the card. I'd add the animation to the card both on hover and on leaving it! 😁

  1

  

  Chris-Ann C.•300

  @cdc2021Posted almost 4 years ago

  @GerbenDol Thank you so much for your feedback :)

  On reflection I can see the extra details that were missing/improvment. All resolved, much appreciated!

  1
• 

  JK•345

  @JakubKepakPosted almost 4 years ago

  Hi Chris-Ann C. ! Nice work 👏

  To centre all content to the centre of a page a would set display:flex to the body and then add justify-content: centre and align-items: centre. In order to work, you would also need to set the property height on the body element. For instance height:100vh. Also, I would maybe set max-width proepty to .wrapper in order not to fill all space in wide screens.

  Anyway, nice job! Keep improving 🙌

  1

  ---

  

  Chris-Ann C.•300

  @cdc2021Posted almost 4 years ago

  @JakubKepak Thank you very much for your feedback, much appreciated :)

  0