time-tracking-dashboard

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi @mohammadT9030,

  Congratulations 🎉 for completing this challenge, the dashboard looks great and is responsive. I particularly like how you used the CSS Grid. I do have a suggestion about your code.

  I took a look at your code and noticed that your HTML contained all of the data which is quite time consuming and long to have all of that data. Another approach you could was the JSON file. After linking the JSON file to the JavaScript file, you can use a for loop to go through each box and insert the data. Also, regarding the duration section (daily, weekly, monthly), you could give it an .active class and whichever is active/clicked on, then that particular button will have the color property of white. This will reduce the need to write repetitive code.

  Overall, great project and wish you the best for your future projects 👍.

  Marked as helpful

  0