Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[datetime2] feat(DateRangeInput3): simpler formatting & parsing API #6456

Merged
merged 4 commits into from
Oct 11, 2023

Conversation

adidahiya
Copy link
Contributor

@adidahiya adidahiya commented Oct 11, 2023

similar to the same change for DateInput3: #6398

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

  • feat(DateRangeInput3): formatDate and parseDate props are now optional. The component will use a date-fns formatter & parser with a format string inferred from the time precision setting.
  • feat(DateRangeInput3): new dateFnsFormat prop allows users to easily customize the date formatter, parser, and input placeholder without having to write custom formatting & parsing functions
  • fix(DateInput3): forward locale to DatePicker3 correctly so that month & year dropdown labels are localized

Reviewers should focus on:

Test cases, no regressions in example

Screenshot

2023-10-11 12 21 17

@adidahiya
Copy link
Contributor Author

[datetime2] fix(DateRangeInput3): support 'locale' prop correctly

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya
Copy link
Contributor Author

[datetime2] fix(DateInput3): forward locale to DatePicker3 correctly

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya
Copy link
Contributor Author

fix DateRangeInput3 locale test

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya adidahiya changed the title [datetime2] feat(DateInput3): simpler formatting & parsing API [datetime2] feat(DateRangeInput3): simpler formatting & parsing API Oct 11, 2023
@adidahiya adidahiya merged commit 42a0c1a into develop Oct 11, 2023
@adidahiya adidahiya deleted the ad/fix-dri3-locale-prop branch October 11, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant