Eugeniaโข 950
@JaneMoroz
Posted
Hey! Congrats ๐ Your solution to this challenge is great! โค๏ธ
The only things I've noticed:
- You need to wrap the main content of the page into the <main> tag. It will solve all these landmark issues and improve accessibility.
- Also when I make it smaller, the card image body is kinda cut in half. You have
height: 100vh;
so I suggest to it changemin-height: 100vh;
and maybe add some top and bottom margins to card. This way you card will still be centred for larger screens and not cut from above for smaller screens.
Keep it up! And good luck ๐
Marked as helpful
0