v6: Update Datepicker and move docs to forms section#42037
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Datepicker component documentation by moving it from the Components section to the Forms section and updates the underlying vanilla-calendar-pro library from v3.0.5 to v3.1.0 for bug fixes and accessibility improvements.
Changes:
- Relocated Datepicker documentation from components to forms section in the sidebar navigation
- Updated vanilla-calendar-pro peer dependency from ^3.0.5 to ^3.1.0
- Added comprehensive new documentation file for Datepicker in the forms section
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| site/src/content/docs/forms/datepicker.mdx | New comprehensive documentation for the Datepicker component with examples, options, methods, and events |
| site/data/sidebar.yml | Moved Datepicker entry from Components to Forms section |
| package.json | Updated vanilla-calendar-pro peer dependency to ^3.1.0 |
| package-lock.json | Updated resolved version and integrity hash for vanilla-calendar-pro to 3.1.0 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
91ba7c5 to
e7ddd9b
Compare
julien-deramond
left a comment
There was a problem hiding this comment.
Thanks for the PR @coliff, LGTM
I updated the changes in .bundlewatch.config.json as we use .25 step, you couldn't know :)
Note for @mdo: there's a difference between Bundlewatch run here by GitHub Actions, and locally with npm run bundlewatch:table. For instance, 8c65e7a fixed locally, but failed with GitHub Actions.
|
Thanks Julien! One thing to consider is the ordering of the pages for forms on the sidebar. They are alphabetically listed for components, helpers and utilities sections (which makes sense), but sort of listed in order of importance/complexity for forms section. Not sure where Datepicker is optimally placed here. |
I’m not entirely sure either. We can revisit this in a follow-up PR once we get Mark’s feedback. One possible approach would be to keep Overview first, then list the remaining pages alphabetically, since they aren’t all components and there isn’t a clear reading order. |
mdo
left a comment
There was a problem hiding this comment.
Placement looks good for now—forms are a little funky like that. I may update ordering elsewhere too—we need to revisit some of the docs navigation I think at this point, too. Something to think about.
Good catch. I’m seeing the same issue when running |
|
sorry for missing that issue! I can reproduce now. |
|
Fixed the issue! Needed to move the grid styles to the new Moving |
Update Datepicker and move docs to forms section Update .bundlewatch.config.json
02259df to
4f2ac6b
Compare
Neat! Thank you!
Yes, sure. |

Description
v6: Update Datepicker and move docs to forms section. Change agreed with @mdo.
Vanilla Calendar Pro update includes some useful bug fixes and accessibility improvements: https://github.com/uvarov-frontend/vanilla-calendar-pro/releases/tag/v3.1.0
Type of changes
Checklist
npm run lint)Live previews
Related issues