feat(DatePicker): New DatePicker component#3283
Conversation
|
View your CI Pipeline Execution ↗ for commit a19fc7b ☁️ Nx Cloud last updated this comment at |
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
dreamwasp
left a comment
There was a problem hiding this comment.
this is the default, i noticed when i navigated dates in the Calendar story the second month doesn't change (it stays March the whole time)
the DatePicker itself looks way better and i like the simplicity of 'useDatePicker' hook. these are just my first thoughts, i'll take a deeper look at accessibility + structure later today!
| // Pad end of last week with nulls | ||
| if (currentWeek.length > 0) { | ||
| while (currentWeek.length < DAYS_PER_WEEK) { | ||
| currentWeek.push(null); | ||
| } | ||
| weeks.push(currentWeek); | ||
| } |
There was a problem hiding this comment.
what are these null returns for? days left over at the end of the month?
| return disabledDates.some((d) => isSameDay(date, d)); | ||
| } | ||
|
|
||
| /** |
There was a problem hiding this comment.
i don't really understand what this means - clamp?
| boxShadow: '0 4px 16px rgba(0, 0, 0, 0.12)', | ||
| width: 'max-content', | ||
| }) | ||
| ); |
There was a problem hiding this comment.
i found this confusing since this just a wrapper and DatePickerCalendar is the actual calendar
There was a problem hiding this comment.
yeah this can be prettier. DatePickerCalendar is for DatePicker, but in theory the inside calendar parts could be used on their own
@dreamwasp lol Calendar is only supposed to show the one month, the header clearly needs some work. thanks for the callout! |
|
📬 Published Alpha Packages: |
|
🚀 Styleguide deploy preview ready! Preview URL: https://69b96dd3e3cc853fffb9a86a--gamut-preview.netlify.app |


Overview
WIP of the new DatePicker component. Looking for some early feedback, comments, questions, concerns. I left some comments in the code of things I'm questioning.
Things I know are missing/not completely working:
PR Checklist
Testing Instructions
Don't make me tap the sign.
PR Links and Envs