Profile card component solution

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Good job Rafa!

  The truth is that I see everything well done, the design, methodology, layout and scheme is all good.

  The comment of @Ahmed Bayoumi is very good to add more semantics to your project.

  I only add that you should investigate about normalize CSS and reset CSS, since they are more useful for resetting styles in browsers instead of using the universal selector '*' in your CSS because it can reduce the performance of the page.

  By the way you have improved, keep it up :D!

  Marked as helpful

  2

  ---

  

  Rafael Takano•180

  @rafaeltakanoPosted over 2 years ago

  @Sdann26 Hey there! Thank you for the feedback!

  I read about it and I'll use reset CSS on my next projects to see how it goes

  Thank you for teaching me something new!

  1
• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Rafael, It looks great... Your code is so clean, 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

  2

  ---

  

  Rafael Takano•180

  @rafaeltakanoPosted over 2 years ago

  @Bayoumi-dev Thank you for your feedback!

  I'm trying to learn how to structure my html in a better way and this helped a lot!

  0