Completed this challenge using very basic css, responsive css

Community feedback

• 

  Chow Jia Ying•145

  @C-likethis123Posted over 3 years ago

  In general, I like your code organisation and how clean your code is.

  Some things I have noticed:

  • On screen sizes below 830px, some of the text tends to overflow out of the container. Instead of setting a fixed height, you could set it to height: fit-content.
  • The size of the statistics in your solution seems to be smaller than the original design
  • You have some code that specifies two font weights like this: font-weight: 400, 700. I'm not sure whether it works on your side, but in my browser it's flagged as an invalid property value.
  • The words (particularly the description) in your solution seems to be less spaced out than in the original design. You can add a line-height: 20px to increase the spacing between lines.

  1

  ---

  

  Nishkarsh Dubb•165

  @Nishkarsh01Posted over 3 years ago

  @C-likethis123 Notes have been taken, thanks :)

  0