Travolgi 🍕• 31,420
@denielden
Posted
Hi Luiza, good job! I took some time to look at your code and have some ideas for improving it:
- add
main
tag and wrap the card for improve the Accessibility - add descriptive text in the
alt
attribute of the image - remove
div
withcontainer
class because it is superfluous - remove all
margin
fromcard
class - try to use flexbox to the body for center the card. Read here -> best flex guide
- after, add
min-height: 100vh
to body because Flexbox aligns child items to the size of the parent container
Overall you did well 😉
Hope this help and happy coding!
Marked as helpful
0
Luiza Moshkin• 70
@la-viza
Posted
Hi Deniel! Thanks for your much appreciated help – your suggestions worked. Cheers!
1