-
Notifications
You must be signed in to change notification settings - Fork 25
fix(DateInput): onChange trigger #5324
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
base: beta
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: f7e034b The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## beta #5324 +/- ##
==========================================
- Coverage 87.96% 87.62% -0.35%
==========================================
Files 227 227
Lines 7772 7973 +201
Branches 3406 3549 +143
==========================================
+ Hits 6837 6986 +149
- Misses 935 987 +52
... and 44 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
packages/form/src/components/DateInputField/__tests__/index.test.tsx
Outdated
Show resolved
Hide resolved
262f5a9
to
0dff9a0
Compare
b7af3bd
to
d25d9d8
Compare
949e180
to
fcbdf9a
Compare
Summary
Type
Summarise concisely:
DateInput
: value changes onBlur instead of onChange to avoid wrong dates while the user is typing