Time Tracking Dashboard Using HTML | CSS | JS

Community feedback

• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  Congratulations on completing another challenge, it's looking great! I have looked into your code and have few suggestions:

  1. "Report for Jeremy Robson" should all be enclosed by a h1 and each title should be an h2.

  2. There must be a way to tell the user which button is active, i.e. daily, weekly or monthly stats, at the moment. You can use aria-expanded or aria-selected on button.

  3. Add some active and hover states on button to make them standout when any of them is selected, you can use aria-selected="true" on the button which is active and then apply the color using the selector like this [aria-selected=true]{ color: #fff; }

  4. I have tested your solution on smaller heights, you should add some vertical padding on the body as well.

  5. As it can be seen on the initial loading of page, the static data of 0hrs is shown, which doesn't really look nice, you can use a function to load the data on window.onload or something similar in Javascript and keep one button active on loading.

  6. Check the accessibility report and try to correct all the mistakes, it will really help you do things semantically well.

  Good luck!

  Marked as helpful

  0