profile-card-component-main

Community feedback

• 

  Krysia Sherburne•120

  @klduplessPosted almost 2 years ago

  Wow, this looks great!

  To match the challenge design better, I would make a few tweaks.

  It looks like you are missing the drop shadow on the profile card. It's very subtle in the original design so it can be hard to tell!

  .card {
  ....
  box-shadow: [put offset, blur radius, and color here]
  }
  

  I also think the border-radius on the card needs to be a little bit bigger to match the design.

  Lastly, replace the h1 on the name & age, and h2 on the followers, likes, and photos numbers with a span or p element for proper semantic HTML - heading tags should be used for denoting structure, rather than resizing text. More on that here.

  Marked as helpful

  1

  ---

  

  Scott•50

  @Scott-321Posted almost 2 years ago

  @kldupless ,

  Thank you so much for the feedback! Some good tips, especially regarding the use of headings.

  Scott

  0