Time tracking dashboard main Solution(By HTML, CSS, JS)

Community feedback

• 

  Thiago Santos•295

  @thiago-hdsPosted almost 3 years ago

  Hi! Nice work so far!

  1- You can do it by setting the background-color and border-radius on the parent div. It would be something like:

  .work {
    background-color: hsl(15, 100%, 70%);
    border-radius: .83rem;
  }
  

  2- The person card is bigger because all the other cards have width: 90%. I'm not sure why you need to do it.

  Here are some other suggestions:

  • You can set a max-width to the cards in the mobile layout so that they don't stretch all the way across
  • The card icons are only decorative so you can use background-image.
  • There is a lot of duplication in your script. You can try to come up with one function that receives the selected time period as argument and updates the UI. You can create an object to hold all the data and use it in your function.

  Hope this is helpful!

  Marked as helpful

  2