Design comparison
Community feedback
- @henmachucaPosted almost 3 years ago
Hi Filipe! I would try to add the icon inside the image overlay as it seems to be missing from your submission.
Marked as helpful0@flx-lander7Posted almost 3 years ago@henmachuca Eu tentei colocar, mas a parte do posicionamento eu me perdi um pouco. A questão do posicionamento (relative, absolute etc.).
Vou tentar revisar
0@henmachucaPosted almost 3 years ago@flx-lander7 não vi que era br também hehe! Cara, se precisar de alguma ajuda avisa que posso te mostrar como
0 - @denieldenPosted almost 3 years ago
Hi Filipe, great job!
Adding
main
tag for Accessibility and try to remove allmargin
frombody
and use flexbox to the body for center the card. Read here -> flex guideAlso set
heigth
of body to100vh
because Flexbox aligns to the size of the parent container.Hope this help and happy coding :)
Marked as helpful0@flx-lander7Posted almost 3 years ago@denielden Obrigado Deniel. Vou dar um olhada e ajustar.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord