Skip to content

Conversation

@balajis-qb
Copy link
Contributor

@balajis-qb balajis-qb commented Oct 21, 2025

Problem
This PR focus a small accessibility related issue in the DatePicker. When a day is both keyboard selected and mouse hovered, it's color is black which makes it hard to read in it's dark background color. In this PR, I just reset the color back to white for keyboard selected + mouse hovered day.

Changes

  • Reset the style back to white

Screenshots

The Issue
image

After the fix
image

Contribution checklist

  • I have followed the contributing guidelines.
  • I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

- Reset the color to white for accessibility reason
@codecov
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.29%. Comparing base (09af697) to head (aa35b3e).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5955   +/-   ##
=======================================
  Coverage   97.29%   97.29%           
=======================================
  Files          30       30           
  Lines        3470     3470           
  Branches     1457     1472   +15     
=======================================
  Hits         3376     3376           
  Misses         94       94           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@martijnrusschen martijnrusschen merged commit 6a4d653 into Hacker0x01:main Oct 21, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants