Testimonial-grid-section-column

Community feedback

• 

  Felix Criollo•575

  @FelixCriolloPosted almost 3 years ago

  Good job 🚀 🚀

  In .container you could also use grid-template-areas for easier the grid layout For example:

  @media screen and (min-width: 800px)
     .container {
         display: grid;
         grid-template-areas:
               "pur pur gray w1"
               "w2 dark dark w1";
         gap: 16px;
         margin: 0 5% 0 5%;
         padding: 1.2rem;
     }
  

  And then change each .badges(n) adding grid-area: pur; (for example)

  More info about grid-template-areas

  Happy coding :3

  Marked as helpful

  1

  ---

  

  Nnenna Miriam Udefi•300

  @Nnenna-udefiPosted almost 3 years ago

  @FelixCriollo Thank you for the feedback. I'm still read up and make the changes

  1
• 

  Harmanjot Singh•985

  @arkharman12Posted almost 3 years ago

  Areas of improvement:

  • Media query breakpoint
  • Sizing
  • Spacing

  Good luck!

  Marked as helpful

  1

  ---

  

  Nnenna Miriam Udefi•300

  @Nnenna-udefiPosted almost 3 years ago

  @arkharman12 Thank you for the feedback. Pls can you lay emphasis on the media query breakpoint. Thank you🙏

  0
• 

  Abhik•4,840

  @abhik-bPosted almost 3 years ago

  👋 Hello Nnenna Miriam , Your solution looks nice & responsive , Well Done 🤩🤩🤩

  Just a opinion remove the margin from section. Other than that everything looks nice , please keep coding this amazing solutions 👍

  Marked as helpful

  1

  ---

  

  Nnenna Miriam Udefi•300

  @Nnenna-udefiPosted almost 3 years ago

  @abhik-b Thank you for the feedback. I knew that margin was going to be an issue😂. I tried removing it before and it resized the image in the second and fourth sections at the iPad viewport. I'll try to make the changes. Thanks

  1