your-profile-card

Community feedback

• 

  Rohit Deshpande•870

  @rohitd99Posted over 1 year ago

  Hello Rohit, from Rohit 😁

  Congrats for completing the challenge.

  I have but a few suggestions for you.

  1. You have used article as semantic element correctly so I'd recommend using main too instead of the div class="container" since the card is the main content of this page.
  2. Secondly I see you've used div for the background image I'd say you can use background-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