Responsive Social proof section with CSS Flexbox

Community feedback

• 

  Kristina Radosavljevic•380

  @KristinaRadosavljevicPosted about 2 years ago

  Hi and congrats on completing the challenge, it looks pretty good :)

  I did notice a couple of things that you might want to take a closer look at:

  • I would advise against limiting the height of the .testmonial__card, because then if the text gets too long, it's cut off, and if it gets shorter, it leaves a lot of blank space at the bottom of the card.
  • I would consider putting the media query a lot sooner than 425px (or using another breakpoint in between). The layout starts to break as soon as around 1000px.
  • As for your question about using transform: translate(), I personally don't see any problem with using that - it looks fine and it scales well. In my solution, I used margins to push the elements, but I honestly don't see any significant advantage of using one over the other :)

  I hope this helps! :)

  Marked as helpful

  1

  ---

  

  Ssembatya Dennis Duncan•260

  @ssembatya-dennisPosted about 2 years ago

  @KristinaRadosavljevic Thanks a lot and this definitely helps for sure.

  1