Social proof section using HTML, CSS and CSS media query.

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Abbas Saad! 👋

  Nice work on this challenge! 👍

  I suggest,

  • Adding a max-width to the main container or wrapper to ensure that the content of the page doesn't look too stretched on extra-large screens.
  • Centering the content of the page (especially the testimonial boxes so that there's less empty space around the design) in the mobile layout of the screen.
  • Getting rid of the extra space below the last testimonial card in the mobile layout of the site (there's quite a bit, at the moment).

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Abbas Saad•260

  @AbbasSaad27Posted almost 4 years ago

  @ApplePieGiraffe Thanks a lot for the feedback and for pointing those things out!! I'll definitely look into them!! Happy coding :D

  0
• 

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  Great work so far. However, it would be great if you use background-image to set images rather than img element. And about rating boxes and testimonials, I used Grid box. Grid is good for x, y positioning elements while Flex is only for x positioning. Here's my solution. https://social-proof-green.vercel.app/ Hope this helps. Happy coding~ :D

  1

  ---

  

  Abbas Saad•260

  @AbbasSaad27Posted almost 4 years ago

  Thanks a lot for the feedback! I actually tried to use background-image first but had some issues with it. So I went with the img element. Your grid box solution is really awesome and also that animation! I still haven't completely learned grid. I'll start using grid box as soon as I complete learning it. Thanks for the suggestion!

  0