Hi, @Mr-jaw. It's a really good solution. Keep coding. I have some suggestions for you.
1 - You can add min-width:311px
to .container
for the mobile version.
2 - I highly recommend to use one direction margin like margin-bottom
. This way you can avoid margin-collapse and your code will be more readable.
3 - I think you don't need three section
tags here. Because they don't separate main
to three part. They are in the same area. You can use one section
tag and inside three div elements.
4 - You also don't need to give a class name to your main
element in the page. main
element is a unique element in the page like body
. You can add some layout properties to main
with div
.
And congratulations to your 13th Challenge
. I hope I'll reach this point but for now, I can't imagine:), I spend lots of energy to solve these projects, 13th Challenge
is dreamy for me. They are really hard but so beneficial for us.
I hope these will help you to improve your perspective. Keep coding. All the best.
Marked as helpful
@Mr-jaw
Posted
@adonmez04 Thanks for the review 😊
I'll look into the things that you pointed out
Also, good luck on your journey as well