Testimonials grid section

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Tyler Gumb! 👋

  Good work on this challenge! The desktop and mobile layouts look pretty good! 👏

  I suggest,

  • Adding a max-width to the grid container so that it doesn't get too stretched on extra-large screens.
  • Decreasing the space around the sides of the grid so that there's more room for the testimonial cards when the screen width decreases (you can probably do that by adding width: 100% to the grid container, adding a max-width, and adding some margin or padding around the grid container).
  • Switching to a mobile-friendly layout sooner than 395px so that the design doesn't look so squished right before the layout changes from desktop to mobile. Including a tablet layout of the grid might be a nice idea.

  Your code looks pretty neat and organized (e.g., you're making nice use of BEM for your class names). Kudos for that! 👍

  Keep coding (and happy coding, too)! 😁

  2

  ---

  

  positivevibes•205

  @posivibezPosted almost 4 years ago

  Thank you so much this is fantastic feedback! Much appreciated.

  0