Skip to content

fix(web): raise month picker day contrast in dark theme - #3427

Merged
tyler-dane merged 1 commit into
mainfrom
cursor/monthpicker-dark-contrast-545d
Sep 5, 2026
Merged

fix(web): raise month picker day contrast in dark theme#3427
tyler-dane merged 1 commit into
mainfrom
cursor/monthpicker-dark-contrast-545d

Conversation

@tyler-dane

@tyler-dane tyler-dane commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

What and why

The sidebar month picker painted unselected day numbers as inherited black (color: inherit from react-datepicker's #000). That met contrast on Light Beach and failed on Dark Abyss, where the numbers sat on the near-black sidebar. Unselected days and weekday labels now use theme text (--text / --text-muted). The selected week capsule still uses --on-accent on --accent.

The datepicker contrast e2e now runs in both themes so this cannot regress on dark only.

Month picker in dark theme after the contrast fix

monthpicker_dark_contrast_after.mp4

Verify

VERDICT: PASS

Checks run: test:web, type-check, lint, knip, test:a11y, test:e2e

Open in Web Open in Cursor 

Sidebar days inherited react-datepicker's black text, which met contrast
on Light Beach and failed on Dark Abyss. Paint unselected numbers with
theme text and cover both themes in the datepicker a11y check.

Co-authored-by: Tyler Dane <tyler-dane@users.noreply.github.com>
@tyler-dane tyler-dane added the agent-automerge Agent-loop finished; merge-guard may squash-merge label Sep 5, 2026 — with Cursor
@tyler-dane
tyler-dane marked this pull request as ready for review September 5, 2026 18:24
@tyler-dane
tyler-dane added this pull request to the merge queue Sep 5, 2026
Merged via the queue into main with commit c2dfe8e Sep 5, 2026
32 checks passed
@tyler-dane
tyler-dane deleted the cursor/monthpicker-dark-contrast-545d branch September 5, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-automerge Agent-loop finished; merge-guard may squash-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants