Chow Jia Ying• 135
@C-likethis123
Posted
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
@Nishkarsh01
Posted
@C-likethis123 Notes have been taken, thanks :)
0