time-tracking-dashboard with pure html, css and js

Community feedback

• 

  Jay Khatri•4,250

  @khatri2002Posted about 2 months ago

  Hi @opeoluwa7!

  The developed solution looks great! The responsiveness is well-handled! Great job on that.

  Here are a few suggestions for improvement:

  ---

  1. You've used an li element to render the text "Daily," "Weekly," and "Monthly" and directly attached the onClick event to it. While this works functionally, a button element would be more appropriate since it is interactive and triggers an action. Using a button improves accessibility, allows for better keyboard navigation, and provides built-in browser behavior such as focus states.

  2. The hover effects seem to be missing for buttons and cards. Consider adding them as per the design reference, as hover effects enhance user experience by providing visual feedback and making interactions feel more intuitive.

  3. You've set cursor: grab; for both cards and buttons. While this is useful for draggable elements, a pointer cursor would be more appropriate for buttons and clickable cards. cursor: grab; is typically used when an element can be dragged, indicating that the user can move it. On the other hand, cursor: pointer; is best suited for interactive elements like buttons and links, where clicking triggers an action.

  ---

  The rest of the solution looks great! Keep up the excellent work! 🚀

  Marked as helpful

  0