Profile Card Component

Community feedback

• 

  Arturo Simon•1,785

  @artimysPosted almost 4 years ago

  Nice work Soodmax, it looks close to the design. I have a few minor suggestions:

  • I'm not sure if the "Followers", "Likes", and "Photos" list warrants to be in a <nav> but if it does I recommend anchor tags in your <li>
  • double check on the font-color for p.name and London <p> tags
  • font-color also looks really light for "Followers", "Likes", and "Photos"
  • Be sure to add text to an image's alt attribute for accessibility

  Keep it up and have fun coding!! 👍

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, good job on this challenge, soodmax! 🙌

  Your solution looks good! 👍

  I suggest using the <strong> tag rather than the <b> tag to make the name in the title of the card bold—apparently, it's better for semantic meaning and a little less confusing in its original use than <b>. 😉 Also, giving your page a title with the <title> tag would be a nice touch!

  Keep coding (and happy coding, too)! 😁

  0