My submission for Testimonial Grid Sections

Community feedback

• 

  Alex•660

  @al3xbackPosted almost 3 years ago

  Hi Quang,

  Would be good if we dont set width on each grid item (in this case on .card class) if we use grid-template-areas.

  let the parent controls the width of its children using grid-auto-columns: 1fr; or grid-template-columns: repeat(5, 1fr);

  Marked as helpful

  0
• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Quang, first of all very well done you completed this challenge. Here I have noticed one issue in your work.

  1. Your design is not aligned center to screen/viewport. Solution:

     Change margin: 3rem; To margin: 3rem auto It will solve your alignment problem.

  Overall, it is really good, you also made it responsive. Great

  Good Luck

  0

  ---

  

  Quang Hung Pham•470

  @pqhung3007Posted almost 3 years ago

  @skyv26 A minor but worthy change. Thanks for correcting me Aakash!

  1