@elaineleung
Posted
Hi Aman, great work here! I think you've done really well with the responsiveness, and the placement of the cards in the tablet view was quite an interesting choice (which I think I like!). Some quick observations for you:
-
The font weight for the paragraph under the main heading might be a bit too light, so you might want to check the reference again. It should be the same font weight as the text in the cards.
-
In the desktop view, the "calculator" and "karma" cards in your solution seem to be in a different placement than the ones in the design.
-
You're using flexbox on the body for centering, which is great; the only thing you're missing is the
align-items: center
property because right now, theflex-direction
isrow
(by default if you don't declare it specifically), and so the vertical alignment in that case would be done viaalign-items
Well done on the whole! π
Marked as helpful
@AmanGupta1703
Posted
@elaineleung Thank you! π