Rohit Deshpande• 870
@rohitd99
Posted
Hello Rohit, from Rohit 😁
Congrats for completing the challenge.
I have but a few suggestions for you.
- You have used
article
as semantic element correctly so I'd recommend usingmain
too instead of thediv class="container"
since the card is the main content of this page. - Secondly I see you've used
div
for the background image I'd say you can usebackground-image
property on your body for the same, since those images don't have any meaning this should be better. Here's a video from Kevin Powell that shows you how to use multiple images as background.Video
Hope it helps
0