Profile Card Component

Community feedback

• 

  Darwin Grisales•130

  @jhonGriGiPosted almost 2 years ago

  In general, this is a good solution; be careful with the extra double quote in your index.html line 16.

  My recommendation for Sass and Bem is:

  1: Instead of using your html tag in CSS, use a class or id attribute; this way, if you want to change the attribute, you don't have to change the CSS files.

  from: hr: { // code }

  to: .separator { // code }

  2: I would rename the .stats__text and .stats__bold classes to .stats__text-bold and .stats__text-normal to make your code easier to read. 

  Good job with your solution.

  Marked as helpful

  1