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

Allow clear dates to clear partially entered dates #554

Conversation

timhwang21
Copy link
Collaborator

Addresses #525.

Move user input state out of <DateInput/> and into either <DateRangePickerInputController/> or <SingleDatePicker/>, passing down the value and a setter as props.

Copy link
Member

@ljharb ljharb left a 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.

@ljharb ljharb added the semver-major: breaking change A non-backwards-compatible change; anything that breaks code - including adding a peerDep. label Jun 10, 2017
@timhwang21
Copy link
Collaborator Author

timhwang21 commented Jun 11, 2017

Agreed, I couldn't think of anything better that differentiated from the existing variable inputValue.

@timhwang21 timhwang21 force-pushed the timhwang21/525-clear-partial-dates branch from b79eea1 to b6d1408 Compare June 30, 2017 02:16
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 86.263% when pulling b6d1408 on timhwang21:timhwang21/525-clear-partial-dates into c96a237 on airbnb:master.

@timhwang21
Copy link
Collaborator Author

@majapw rebased this, #555, and #580, please let me know if there's anything else I can do for these PRs!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 86.374% when pulling 8eb1ebd on timhwang21:timhwang21/525-clear-partial-dates into 83d55bf on airbnb:master.

@timhwang21 timhwang21 force-pushed the timhwang21/525-clear-partial-dates branch from 8eb1ebd to 0ab40c2 Compare August 10, 2017 13:18
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.
@timhwang21 timhwang21 force-pushed the timhwang21/525-clear-partial-dates branch from 0ab40c2 to e10c0a7 Compare August 10, 2017 13:21
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 86.599% when pulling e10c0a7 on timhwang21:timhwang21/525-clear-partial-dates into 9685cc9 on airbnb:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 86.599% when pulling e10c0a7 on timhwang21:timhwang21/525-clear-partial-dates into 9685cc9 on airbnb:master.

@jrparish
Copy link
Contributor

We are running into #525 and would love to have this change merged. Is there any reason we're holding on this PR?

@timhwang21 timhwang21 closed this Jun 11, 2024
@ljharb ljharb reopened this Jun 11, 2024
@timhwang21 timhwang21 closed this Jun 11, 2024
@timhwang21 timhwang21 deleted the timhwang21/525-clear-partial-dates branch June 11, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-major: breaking change A non-backwards-compatible change; anything that breaks code - including adding a peerDep.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants