Skip to content

Conversation

DRiFTy17
Copy link
Collaborator

@DRiFTy17 DRiFTy17 commented Jul 8, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added/updated: Y
  • Docs have been added/updated: Y
  • Does this PR introduce a breaking change? N
  • I have linked any related GitHub issues to be closed when this PR is merged? N

Describe the new behavior?

Added a new dateFormat property (date-format attribute) to allow for developers to use a preset built-in alternative date format. This reduces the burden on developers to use custom parse and format callbacks, and avoids complexity related to the input mask and value coercion logic.

Additionally because the built-in "n" and "t" keyboard shortcuts for setting the "today" date value can conflict with formats that use the MMM block, we will automatically adjust the shortcuts when those formats are chosen. A new shortcuts property/attribute has also been added to allow for turning off the shortcuts manually, or to provide custom single character shortcuts.

@DRiFTy17 DRiFTy17 requested a review from a team as a code owner July 8, 2025 18:26
@DRiFTy17 DRiFTy17 added minor Increment the minor version when merged storybook-preview labels Jul 8, 2025
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Contributor

github-actions bot commented Jul 8, 2025

View Storybook Deployment
pr/tyler-technologies-oss:feat-date-picker-alt-formats

Copy link
Contributor

@samrichardsontylertech samrichardsontylertech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks good to me. I'm a little split on shortcuts because it could also be handled by the consumer with an event listener on the <input>, but similar libraries include hooks like this and I generally appreciate those. It's also probably better for consistency to handle the actual event from our code.

Copy link
Contributor

@Ross-Blakeney Ross-Blakeney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. I don't see any issues.

@DRiFTy17 DRiFTy17 merged commit 7cd614e into main Jul 15, 2025
15 of 16 checks passed
@DRiFTy17 DRiFTy17 deleted the feat-date-picker-alt-formats branch July 15, 2025 12:15
Copy link
Contributor

🚀 PR was released in v3.10.0 🚀

@github-actions github-actions bot added the released This issue/pull request has been released. label Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released This issue/pull request has been released. storybook-preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants