Testimonials Grid System

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi there! 👋

  Nice work on this challenge! 👍

  There are some areas that can be improved.

  • I suggest adding a max-width to the container to prevent the cards from becoming too large at screen size below 1200px. In my opinion, a 700px max-width would be good enough.
  • For the quote, I recommend wrapping the p element with blockquote for better semantic HTML.
  • My final suggestion is to write the styling using the mobile-first approach. It often leads to shorter and better performance code. Also, mobile users won't be required to process all of the desktop styles.

  That's it! Hope this helps.

  Marked as helpful

  1

  ---

  

  Obaida Shurbaji•250

  @obaidash99Posted over 2 years ago

  Thank you a lot sir! @vanzasetia

  I will start using the mobile-first approach soon, I'm just getting started with coding on my own and your comments are a huge help!

  Waiting for your opinions about my future projects!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @obaidash99 You're welcome! 👍

  1