@vanzasetia
Posted
Hello, Gabriela! 👋
Good effort on this challenge! 👍
I recommend removing the min-width: 100vh
from the body
element. It makes the site has a horizontal scrollbar on mobile view. Also, it's best to never set any width
into the body
element. You want to think of the body
element like a paper so you should not control the width
of the body
element instead control the size of the elements inside it.
For the card, I highly suggest only setting a max-width
(not a width
). By doing this, you prevent the card from becoming too large on a wide screen and letting it shrink on tiny screen sizes.
The alternative text for the image shouldn't be hyphenated (like code). It should be human readable. Also, alternative text for images should not contain any words that are related to "image". It's already an image element so there's no need to repeat the information.
That's it! Hope this helps. 😊
Marked as helpful