Profile Card Component

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Rakesh, Congratulations on completing this.

  • It looks good! but you have some accessibility issues you need to fix.
  • Document should have one main landmark, Contain the component in <main>.

  <main>
     <div class="container">
        //...
     </div>
  </main>
  

  • Page should contain a level-one heading, Change h2 to h1 You should always have one h1 per page of the document.
  • I suggest you put the status of the profile card in the list item for Semantic HTML like this:

  <ul class="stats">
     <li><span class="stats-num">80K</span>Followers</li>
     <li><span class="stats-num">803K</span>Likes</li>
     <li><span class="stats-num"> 1.4K</span>Photos</li>
  </ul>
  

  I hope this is helpful to you... Keep coding👍

  Marked as helpful

  0

  ---

  

  Rakesh Das•90

  @Rakesh4688Posted over 2 years ago

  @Bayoumi-dev Thanks.❤️ I will change it according your suggestions.

  0