CSS Grid - JS Data Fetchting

Community feedback

• 

  Jay Khatri•4,250

  @khatri2002Posted 2 months ago

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

  Suggestion:

  You’ve used the <div> 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 <div>?

  • Semantics and Accessibility:

    • <button> is designed for user interaction and clearly conveys its purpose to assistive technologies (like screen readers).

  • Consistency in Design:

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

  Switching to <button> ensures your solution adheres to web standards, improves accessibility, and provides a more polished user experience.

  ---

  Fantastic job so far! Keep it up! 🚀

  0