Profile-Card

Community feedback

• 

  Aditya Singh•460

  @adityas24Posted about 2 years ago

  Hi Saswat, great effort by you!

  I saw that for displaying the person's image, you are using background image. But the image tag is the most appropriate way to display image here. For more information read this stackoverflow answer on background image vs img tag

  Hope it was helpful, Happy coding!

  Marked as helpful

  1

  ---

  

  Saswat Samal•80

  @saswat10Posted about 2 years ago

  Thanks, the article was helpful. I'll make the necessary changes and post a new solution

  0
• 

  Favour•2,160

  @Nadine-GreenPosted about 2 years ago

  HEY SASWAT!

  About your issue with centering, a quick fix for this would be the give the body a height of 100vh height:100vh then a display of grid display:grid and then using the code place-items:center

  IF THIS WAS HELPFUL IN ANYWAY, DON'T HESITATE TO MARK IT AS HELPFUL :)

  HAPPY CODING!

  Marked as helpful

  1

  ---

  

  Saswat Samal•80

  @saswat10Posted about 2 years ago

  Thanks made the necessary changes. It worked👍

  0

  

  Favour•2,160

  @Nadine-GreenPosted about 2 years ago

  @saswat10

  Happy that i could help

  0
• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • To not only improve your HTML code but to identify the main content of you page, you will want to wrap your entire component inside the main element.

  More Info:📚

  MDN Main Element

  • “Victor Crest” is a heading. So it needs to be wrapped in a Heading Element.

  • The statistics are a list. They need be created using the Unordered List along with the List Element.

  More Info:📚

  MDN <ul>: The Unordered List element

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  1