-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(DatePicker): add override behavior for start date deletion bug #13443
Conversation
✅ Deploy Preview for carbon-components-react ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Oooh I was gonna bug you to look at this next week, but you're too fast! 🔥 This works if I type in the date and click off, but if I click a date on the calendar and click off it disappears, or click back in/out if disappears too. wondering if that is also something we could/should fix? |
Ah I see what you mean... I/we can take a look next week! |
…980-bug-date-picker-of-type-range-never-call-onchange-when-second-input-value-change
Co-authored-by: Alison Joseph <alisonjoseph@users.noreply.github.com>
Ready for re-rev! @alisonjoseph @andreancardona |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
…ll-onchange-when-second-input-value-change
…ll-onchange-when-second-input-value-change
Closes #12980
Changelog
Changed
fixEventsPlugin
to programatically re-set the start date that is getting deleted on theendDate
blur due to the flatpickr bugTesting / Reviewing
Follow reproduction steps in #12980 and confirm calendar gets updated in PR deployment. Also test DatePicker keyboard and mouse behavior to ensure no regressions