-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Allow clear dates to clear partially entered dates #554
Allow clear dates to clear partially entered dates #554
Conversation
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.
the code LGTM, but i'm not really a fan of the "user" prefix on names.
Agreed, I couldn't think of anything better that differentiated from the existing variable |
b79eea1
to
b6d1408
Compare
8eb1ebd
to
0ab40c2
Compare
Addresses react-dates#525. Move user input state out of `<DateInput/>` and into either `<DateRangePickerInputController/>` or `<SingleDatePicker/>`, passing down the value and a setter as props.
0ab40c2
to
e10c0a7
Compare
We are running into #525 and would love to have this change merged. Is there any reason we're holding on this PR? |
Addresses #525.
Move user input state out of
<DateInput/>
and into either<DateRangePickerInputController/>
or<SingleDatePicker/>
, passing down the value and a setter as props.