Responsive landing page using CSS Grid

Community feedback

• 

  dewslyse•3,025

  @dewslysePosted about 3 years ago

  Hello @yasser22269. Congrats on your submission. Your page is very responsive. Some things to note:

  • You should set a max-width on the main to prevent the content from spanning the full width of the page on large devices.
  • I think it's easier to use grid-template-areas to position the individual testimonial cards on large devices. Your approach of .cards-left and .cards-right is causing a disparity in height between kira's card and the rest.

  Marked as helpful

  1

  ---

  

  yasser Abd Elghany•165

  @yasser22269Posted about 3 years ago

  @dewslyse Thank you very much, I made the adjustments you told me and now it is much better than before, and you can see it now and thank you

  1

  

  dewslyse•3,025

  @dewslysePosted about 3 years ago

  @yasser22269 It looks much better now. If it matters to you, you could center the whole testimonial grid to the page. It is currently tucked to the left.

  Marked as helpful

  1