css Grid, html, flexbox

Community feedback

• 

  Marius•190

  @MariusHorPosted over 2 years ago

  Great job so far! 🙂

  for the background you could add these lines: background-color: hsl(225, 100%, 94%) background-size: contain background-repeat: no-repeat.

  I would also suggest that you dont set fixed heights and widths on elements. Use padding to create space within an element and margins to create space between elements and not top left or position: relative / position: absolute. Google "spacing in CSS" for a better understanding.

  Keep going, you are getting there! 🙂

  0

  ---

  

  Eric Wisdom Agu•20

  @EricwizPosted over 2 years ago

  @MariusHor thanks so much for the correction I really appreciate!

  0
• 

  clive•140

  @clivepato93Posted over 2 years ago

  Hi there for your background-repeat use no repeat

  0