Social proof section using Sass, CSS grid & flexbox layout

Community feedback

• 

  Anna Leigh•5,135

  @brasspetalsPosted about 4 years ago

  Hi, Natsuki! Congrats on completing another challenge!

  I wanted to let you know there's a much easier way of handling the background images. You can actually set multiple background images, which is very handy. For this challenge, I would suggest putting both in the body like this:

  background-image: url("./images/bg-pattern-top-desktop.svg"), 
  url("./images/bg-pattern-bottom-desktop.svg");
  background-position: top left, bottom right;
  background-repeat: no-repeat, no-repeat;
  

  This way you also don't need to set a specific height value on your containers in order to get the bottom image where you want it. Hope this helps!

  1

  ---

  

  Nafsuki•245

  @NafsukiPosted about 4 years ago

  @brasspetals Thank you very much for your feedback! It helps me a lot! I'll work on this challenge again with your solution! :)

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Natsuki! 👋

  You've done a good job on this challenge! Your solution actually responds quite well (something people sometimes have trouble with in this challenge)! 😊

  I like your decision to use CSS grid to layout the boxes with the SVG stars. 😉

  Keep coding (and happy coding, too)! 😁

  0

  ---

  

  Nafsuki•245

  @NafsukiPosted almost 4 years ago

  @ApplePieGiraffe Thank you very much for your kind & constructive feedback!! It motivates me a lot:)) Looking forward to the next challenge!!

  0