Travolgi 🍕• 31,420
@denielden
Posted
Hi Lucas, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
img
element must have analt
attribute, it's very important!- to make it look as close to the design as possible set
width: 100%;
to.img-main
class and removeheight
- remove all
margin
fromcontainer
class because with flex they are superfluous - add flexbox
align-items: center
property to themain
tag for center the card horizontally - after, add
min-height: 100vh
tomain
tag because Flexbox aligns child items to the size of the parent container - try to add a little
transition
on the element with hover effect
Overall you did well 😉
Hope this help and happy coding!
Marked as helpful
1
Lucas Bispo• 50
@LucasBispoMenezes
Posted
@denielden Thank you very much friend, I will study a little more about each point presented in your feedback, to improve both this project and future projects :rocket looks good.
1