Time tracking dashboard

Community feedback

• 

  Jay Khatri•4,250

  @khatri2002Posted 3 months ago

  Hi! The developed solution looks good! The responsiveness is handled very well! Great job!

  You’ve used the <p> element for the Daily, Weekly, and Monthly texts. However, since these are clickable and trigger some action (e.g., switching views), a <button> element would be more appropriate.

  Why Use <button> Instead of <p>?

  1. Semantics and Accessibility:

     • <button> is designed for user interaction and clearly conveys its purpose to assistive technologies (like screen readers).
     • <p> is intended for displaying plain text, not for interactivity. Using it for clickable elements may confuse accessibility tools, making your solution less inclusive.

  2. Built-In Keyboard Support:

     • Buttons are natively focusable and can be triggered using the Enter or Space keys, enhancing usability for keyboard users.
     • Adding similar functionality to a <p> requires extra JavaScript to handle focus and key events.

  3. Consistency in Design:

     • Using a <button> aligns with best practices, as it’s consistent with other interactive elements on your webpage.

  ---

  Fantastic job so far! Keep it up! 🚀

  Marked as helpful

  0

  ---

  

  BRAHIM AJGAGAL•820

  @AslamtoIbrahimPosted 3 months ago

  @khatri2002 Hi I am glad you read my code and give me this precious feedback so thank you so much.

  1