feat(DateRangePicker): add XDSDateRangePicker component - #2201
Conversation
|
@czarandy must be a member of the Meta Open Source team on Vercel to deploy. Learn more about collaboration on Vercel and other options here. |
76508ed to
818a1aa
Compare
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR Analysis Report📚 Storybook PreviewView Storybook for this PR 🧪 Sandbox PreviewView Sandbox for this PR No new or modified components detected. Bundle Size Summary
Accessibility AuditStatus: No accessibility violations detected. Generated by PR Enrichment workflow | Storybook | Sandbox | View full report |
cixzhang
left a comment
There was a problem hiding this comment.
Good implementation — a11y, composition with XDSCalendar, and mobile font floor are all solid.
One gap for API parity: missing changeAction and isLoading. XDSDateTimePicker and XDSDateInput both support these for async optimistic updates. DateRangePicker should have them too for consistency across date input components.
818a1aa to
2f7111c
Compare
|
Added changeAction and isLoading |
2f7111c to
4dbdf3d
Compare
Button-triggered date range picker with a dual-month calendar popover and configurable preset sidebar. Uses XDSCalendar in range mode for selection with hover preview. Value is DateRange | null.
4dbdf3d to
0948d13
Compare
cixzhang
left a comment
There was a problem hiding this comment.
changeAction and isLoading added. Good for parity.
Summary
XDSDateRangePicker, a button-triggered date range picker with dual-month calendar popover and configurable preset sidebar (closes XDSDateRangePicker #184)XDSCalendarin range mode for selection with hover previewpresetsprop with{label, getRange()}— sidebar highlights the active presetDateRange | null({start: ISODateString, end: ISODateString})hasCleardefaults totrue(standard for filter UIs)Test plan
Core/Inputs/DateRangePicker