Testimonials grid section using CSS Grid and SCSS

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi there!

  I have some feedback for this solution:

  • Instead of keep adding body main and etc. I recommend to select directly the class name rather than keep increasing specificity, like this:

   .testimonials {
   /* some styles */
  }
  
  .testim-box1 {
   /* some styles */
  }
  

  • I recommend to fill the alt with their name rather than avatar.

  That's it! Hopefully this is helpful!

  Marked as helpful

  1
• 

  Adriana Eka Prayudha•160

  @radenadriPosted over 3 years ago

  Nice work man! It works greatly on tablet and mobile as well! 🙌

  From what i see, i think i only have two additions based on my personal opinions :

  1. It will be much more awesome if you add the box-shadow on each testimonials, just a thin shadow to make it look exactly like the design. But overall its great, its only little additions as well. 😊
  2. I believe the provided design showing that the avatar is kind of small, if you change it to approximately 45px, i think it will be much more great since it will be much more clearer to see the picture.

  That's it, great work for sure! Happy coding and also keep coding man! Waiting for another work! 👏👌

  Marked as helpful

  1

  ---

  

  dngtnv•190

  @dngtnvPosted over 3 years ago

  @radenadri Wow, thanks for the feedback, i'll try to change that.

  0
• 

  mabrur•105

  @mabrurherzaPosted over 3 years ago

  Awesome 😮 definitely will try this challenge.

  0

  ---

  

  dngtnv•190

  @dngtnvPosted over 3 years ago

  @mabrurherza Thank you 😄, hope to see your solution to this challenge🎈🎈

  0