@denielden
Posted
Hi Ravindra, great work on this challenge! 😉
Here are a few tips for improve your code:
- add
main
tag and wrap the card for improve the Accessibility - For add the top image in the background just put more specific background properties to the
.container
class:
background: url("../img/pattern-background-desktop.svg") no-repeat top center;
background-size: contain;
background-color: #e0e8ff;
- not use
h4
insidebutton
element - remove all
margin
from.summaryContainer
class because with flex they are superfluous - set
min-height: 100vh
to.container
class insteadheight
- add
transition
on the element with hover effect - instead of using
px
use relative units of measurement likerem
-> read here
Overall you did well 😁 Hope this help!
Marked as helpful