@amalkarim
Posted
Hi Sayyid 👋
Congratulations for completing the challenge!
Why do you feel uncomfortable about the result? It's a great solution! However, let me point out some of the things that could be improved. But this is only my personal view, it's not necessarily the best solution out there.
- Never give
height: 100vh;
to the body. When the browser height is less than the content, it will make some parts of the page hidden and we can't scroll to see them. Change it tomin-height: 100vh;
- Add
object-fit: cover;
to.card__image img
to make sure the image isn't squeezed at any given width. - Change
@media screen and (min-width: 375px) and (max-width: 462px)
to@media screen and (max-width: 462px)
in order to avoid the page design back to desktop mode when the screen width is less than 375px (though I'm not sure people nowadays using screen less than 375px)
That's all. Hope this helps. Happy coding!
Marked as helpful