Profile Card Component

Community feedback

• 

  Zienab Mohammed Abdelmonaem•400

  @zenab12Posted over 2 years ago

  Hey @Ola , Congratulations for completing this challenge

  My suggestions

  • use unordered list instead many divs to make it more semantic :

  instead of this:

  <div class="stats-card">
          <div class="stats-content">
            <p>80K</p>
            <span>Followers</span>
          </div>
          <div class="stats-content">
            <p>803K</p>
            <span>Likes</span>
          </div>
          <div class="stats-content">
            <p>1.4K</p>
            <span>Photos</span>
          </div>
        </div>
  
  

  use this

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

  • don't use height or width but use min-height and min-width instead to make it more responsive .to avoid wrap content outside card if it viewed in small screen

  You make clean code ,semantic and responsive so Awesome and Keep Coding 💻😊

  Marked as helpful

  1
• 

  Oladapo Jesufemi Paul•1,060

  @jesufemiganPosted over 2 years ago

  Awesome work! You can add letter spacing to the span tag in the "stats-content" class and increase the font-size

  Marked as helpful

  1