Time tracking dashboard using SCSS and vanilla JS

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted over 2 years ago

  Hi Vijay, this looks great, well done! I think you just need a bit more padding inside the cards and also to make the border-radius a bit rounder, then your solution could be looking closer to the original.

  For the JS, things also look fairly alright to me; one recommendation I have is to use either textContent or innerText for changing the text content instead of innerHTML because you don't actually have HTML to parse, and it's also generally considered good practice to use those first in this situation where you only have plain text. As a "level-up" challenge, see whether you can add some error handling for your async/await function!

  0