responsive page using CSS grid

Community feedback

• 

  Folarin Akinloye•1,240

  @folathecoderPosted over 3 years ago

  Hi KRISTÝNA!

  Nice implementation and clean CSS code. Kudos!

  I only had the privilege to view your work on mobile device, so I don't have feedbacks on other screen sizes.

  On mobile, the left and right margins on the cards should be smaller so it can look closely similar to the sketch!

  Also the shadow in the cards should be reduced a little. You can do this by adjusting the spread of the shadow!

  Overall you did a good job with CSS Grid and you made proper use of Grid Span.

  I recommend you learn scss when you get comfortable with css, because it will help you write more maintainable and dry code.

  I see your class naming style is almost similar to BEM naming convention , you can check out BEM.... it is quite good for nesting similar classes when writing SCSS.

  Great work! 👍

  3

  ---

  

  Kristýna•40

  @KristynRihoPosted over 3 years ago

  @folathecoder Hello Folarin, thank you very much for your feedback!

  0