@darryncodes
Posted
Hi Slasher,
I think you should be pretty proud of this design, all the elements are there and it looks like a close match.
With a few small tweaks it could be perfect:
- you should add
background-repeat: no-repeat;
background-size: 100% auto;
to your<body>
styles - you could center your design by adding
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
to the body too - you could clear down your accessibility report too if you add
alt-text
to your images and swap<div id="caixa">
for a<div id="caixa">
Hope that helps!
Marked as helpful
@slasher314
Posted
@darryncodes i apreciate your comments and i will use them in the future. But i really dont get the last one, Can you explain It better??
@darryncodes
Posted
oh sorry @slasher314 i forgot to update it: swap <div id="caixa">
for a <main id="caixa">
. You should use a <main>
tag.
Hope that helps π
Marked as helpful
@slasher314
Posted
@darryncodes helps a Lot! Hahahhaa Thank you!