Commit 5b1b7a1
refactor(date-picker): idiomatic Tailwind focus ring + snapshot tests
Align with the add-component skill:
- Replace the inline JS focus-ring handlers (el.style.outline / chromeFocusProps)
with the same `ring` utility Button/inputs use — focus-visible:ring on the nav
buttons + trigger, and :focus:ring (with relative/z-10 so it sits above
adjacent cells) on calendar day cells. No more CSS-in-JS.
- Add structural snapshot tests for DateField, DatePicker (closed), and
Calendar, per the skill's testing convention.
The earlier "no visible focus" was a wrong-attribute bug (data-[focus-visible],
a react-aria attr our cells never emit), not a failure of `ring`; the compiled
utilities are present in dist and identical to Button's.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>1 parent dc0f4bb commit 5b1b7a1
3 files changed
Lines changed: 50 additions & 52 deletions
File tree
- packages/core
- __snapshots__
- src/components
0 commit comments