All comments

• 

  Sukruti Konhar•30

  @sukrutikonharSubmitted almost 4 years ago

  I'd like to know if there is anywhere I can improve on this page.

  

  Testimonals Grid Section

  3

  40
  

  Guilherme Magno•300

  @devMagnoPosted almost 4 years ago

  Hello! 👋 Good job on this challenge!

  The layout looks pretty good and it's responsive as well, good job! I loved the tablets and the mobile layout.

  I suggest removing the max-width: 1440px from the desktop media query and adding a max-width to the container instead, because it creates a weird single-column layout on bigger screens. Looking forward to your changes!

  Happy coding! 👊

  2

  This comment has received 2 repliesShow
• 

  Blessing Peters•290

  @ble-synSubmitted almost 4 years ago

  feedback would be very much appreciated!

  

  Testimonial grid section using html and css

  3

  10
  

  Guilherme Magno•300

  @devMagnoPosted almost 4 years ago

  Good job on the challenge It looks pretty cool! 🤩

  I suggest adding a media query for tablets or smaller desktop devices because the layout gets kinda weird before reaching the 600px media query. In my solution I created a 2-column layout and I thought it looked really cool, so maybe you should try doing that.

  Looking forward to your changes!

  Happy coding! 👊

  0

  This comment has received 1 replyShow
• 

  Carlos Flores•120

  @CFlores9812Submitted almost 4 years ago

  hey if i could get some feedback on the code. Its probably easy to do but i was not able to center the grid container on the page

  

  grid

  1

  20
  

  Guilherme Magno•300

  @devMagnoPosted almost 4 years ago

  Hello! Good joob on the challenge, it really looks pretty good!

  To center the grid container you can easily just style the body element like this:

  body {
    min-height: 100vh;
    display: flex;
    flex-direction: column;
    justify-content: center;
  }
  

  Happy coding bro 👊.

  0