@radriann21
Posted
My advices:
-
Use more semantic tags. The tag that encompasses the card should be "article" by convention and to separate the text from the illustration, use a "section". You don't need to use br for separation, use margin properties.
-
Use reset css (not mandatory but good practice!)
-
Also use CSS variables
-
Use the colors and all the resources that the project brings! It should be as close to the design that is presented. Although you can add a touch of your own design.
-
The design doesn't look good at smaller sizes (320px)
However, the design is quite similar to the one presented and the project looks good!
@Oluwa-Sanni
Posted
Thank you for this feedback @radriann21