Hi, @PiwkoO.
I agree with @pRicard0, your project is almost perfect.
Here are some quick tips.
-
For the mobile version, the
.card
class has 327px in the design file. Maybe you can try 327px instead of 400px. And you don't need anypadding
values in your body. You've already centered yourmain
class inside of the body. You can use this method for your nested elements, and they will be more responsive, but it will take a little time. -
Don't use only the
width
property aspx
orem/rem
etc., it will cause some problems. You can usemax-width
instead. And you can declarewidth:100%
. This will allow your browser to do more calculations. -
You can use the
px
unit to reach your first solution. However, you need to update this measurement unit to more responsive ones likerem
orem
. I highly recommend usingrem
. It gives you more control over your code. -
You don't need to use the
height
property for your content area. They should take their height values from their content. I think you missed someline-height
values and that's why your content area height didn't match the image area height.
I hope these will help you. Keep coding and have a wonderful day.
Marked as helpful
@PiwkoO
Posted
@adonmez04 Thank you for making time to give me feedback