Time tracking dashboard

Community feedback

• 

  Ben•770

  @BenConfigPosted about 3 years ago

  Hey, well done on this challenge, it's a very nice solution.

  For your card icons, you have declared background-repeat: no-repeat; and background-position: right; on each one. Instead you could add those to your .card-icon class and remove them from the individual classes which will save you 10 lines of code.

  Also you can remove width: 100%; from .card-icon because it will take the full width of the parent by default, and text-align: end; doesn't appear to do anything so that can be removed too.

  0

  ---

  

  Lourdes González•150

  @Lourdes84Posted about 3 years ago

  @BenConfig Hi Ben! First of all, thank you very much for your comments. I have tried to put the background properties to the card-icon class, but it has not worked for me so in the end I have put it in a single line to reduce the code:

  Example: background: hsl(15, 100%, 70%) url(../images/icon-work.svg) no-repeat right;

  I have also removed width: 100% and text-align: end from .card-icon. You´re right, It isn´t neccesary. Thank you very much for your advice, they have helped me!

  0