Testimonials Grid using CSS grid

Community feedback

• 

  Benja.min•740

  @BenjaDotMinPosted almost 3 years ago

  Hello there!

  You really do have a great grasp with grid, for your first attempt. Much better than I did!

  However, regarding your question:

  • To your body tag add: height:100vh; display: grid; place-items: center;
  • To your main tag add: max-width: 1440px;

  That should improve the size and positions. You can play around with values to match what you need.

  It wont make a difference, but consider the shorthand syntax for grid-template, to save yourself repeating values and make it easier to read.

  For example: grid-template-columns: 1fr 1fr 1fr 1fr; grid-template-rows: 1fr 1fr;

  Could simply become: grid-template: repeat(2, 1fr) / repeat(4, 1fr);

  Marked as helpful

  1

  ---

  

  Matthew Van Pelt•575

  @Mattvp21Posted almost 3 years ago

  @BenjaDotMin Ahhh! I was wondering why justify content didn’t work! Thanks

  1

  

  Benja.min•740

  @BenjaDotMinPosted almost 3 years ago

  @Mattvp21 Not a problem :)

  0
• 

  P

  David Turner•4,150

  @brodiewebdtPosted almost 3 years ago

  Kevin Powell did a video on CSS Grid using this challenge. You might pick up some tips for that. https://www.youtube.com/watch?v=rg7Fvvl3taU

  Marked as helpful

  1

  ---

  

  Matthew Van Pelt•575

  @Mattvp21Posted almost 3 years ago

  @brodiewebdt Cool! I’ll check it out! Thanks

  0