Four Card Feature Section - Sass

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 4 years ago

  Hi Thomas,

  This looks pretty good. I'd still recommend you revisit it and use css grid - it's so perfect for practicing that, and if you used grid areas I think would be quicker than flexbox.

  I'd add some more space between your cards - can do this with the gap property and lose your margins if you like.

  This would also be more responsive if you leveraged flex properties instead of width. Think flex grow shrink and basis 😉

  I hope these are helpful ideas. Keep on coding!

  (and as I've said to others, if a new challenge everyday is too much - which I think it is - don't be afraid to return to old challenges to refractor and improve as you learn more. That's time we'll spent tol!)

  1

  ---

  

  Thomas Boittin•265

  @tboittinPosted almost 4 years ago

  @grace-snow Hi Grace, thanks for the reply.

  Concerning the 1 project per day, I actually planned to refactor previous projects on the week end, trying to explore better ways to do so. I think it still counts as a project haha.

  So yeah, today I'll use flex-grid on this one :)

  Have a great day !

  0

  

  Thomas Boittin•265

  @tboittinPosted almost 4 years ago

  @grace-snow

  I used flex-grid, the improvement is awesome and the code is so clean!

  Couldn't edit the solution, looks like FrontEnd Mentor is working on the edit page but the preview site is updated.

  1