Time tracking dashboard challenge using typescript

Community feedback

• 

  Mariem Ehab•335

  @MaryEhbPosted almost 3 years ago

  The site looks pretty the same as the design and also you are the only one I reviewed who added the hover style to the three dots icons so great job 👏👏 I just have a few remarks:

  • There are more than one element that has the same id 'divsm' and that is not right because the ids are unique and can't be repeated for several elements. You can use classes instead
  • in the same elements as the previous point I noticed that the number of hours change as the user choice but the text (Last week) stay the same where it should also change between (last day - last week - last month)

  Marked as helpful

  0

  ---

  

  sriram•120

  @sriramshiyamPosted almost 3 years ago

  @MaryEhb thanks you for the review, and thanks for the minor details you mentioned 🤗

  0