Mobile first Design with Sass and ES6

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted over 4 years ago

  Hey, John Norman! 👋

  Good work on this challenge! Your solution looks pretty good and the grid responds rather well. 👍

  A tip or two I might suggest,

  • The grid seems to be a little off-center at certain screen widths below 1600px. I suggest adding min-height: 100vh to the body and then using flexbox to center the grid in the viewport (since I often find it easier and more foolproof than other methods).
  • There's also quite a bit of margin around the grid at over 1600px. You won't need that margin anymore if you use flexbox to center the grid and it'll make your solution a little tighter and without too much extra space, I think. 😉

  Keep coding (and happy coding, too)! 😁

  10
• 

  John Norman•880

  @norman02

  Posted over 4 years ago

  Thank you for the suggestion!

  00