Responsive landing page with Grid and Flex

Community feedback

• 

  Debarshi Polley•240

  @Dpolley99

  Posted 3 months ago

  Hello. This is a great solution. I just have one suggestion to place the container at the centre. Instead of margin-block: 5rem; in the body, I would do this:

  min-height: 100vh

  display: grid;

  place-items: center;

  This would convert the body into a grid and place the testimonials grid container exactly at the centre of the screen instead of putting a fixed margin at the top. The fixed margin might work for the deivce in which you are building it, but this method ensures compatibility in all devices.

  Other than that, everything else looks spot on.

  Do try this and mark this comment helpful if it works for you. Cheers!

  10