@denielden
Posted
Hi Guilherme, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
- add
main
tag and wrap the card for Accessibility - to make it look as close to the design as possible add
background-size: contain
to#main-section
id - try to add hover effect and a little
transition
on the element with hover effect - instead of using
px
try to use relative units of measurement -> read here
Overall you did well :)
Hope this help and happy coding!
Marked as helpful
@guilhermehto
Posted
@denielden yoo, thanks for this amazing feedback Deniel, I'll do another pass tomorrow on this solution and apply them to it