@huyphan2210
Posted
Hi, @Rodrigo-Policarpo
You did a great job with the challenge! I have a couple of suggestions for improvement:
- Using
height: 100vh
on thebody
works for this specific challenge, but I'd recommend switching tomin-height: 100vh
instead. This ensures thebody
can expand if the content grows beyond the viewport height, preventing potential overflow issues and making it more flexible across different devices and content lengths. - I noticed you're already using a
footer
element, which is great! Why not replace the.card
'sdiv
with a<main>
element instead? Since<main>
semantically represents the main content of a page, it could be a better fit than a genericdiv
, and it would make your HTML more accessible and meaningful.
Hope this helps!
Marked as helpful
Hi @huyphan2210 !
First of all, thanks for sharing these suggestions and code improvements!
-
The suggestion about the
height
tag was very useful, since I don't have much practice on how to usemin-max
correctly. -
Also, the suggestion to replace the elements was very important too.
Thank you for everything!