-
Notifications
You must be signed in to change notification settings - Fork 860
Add clear functionality to EuiDatePickerRange #8428
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
Add clear functionality to EuiDatePickerRange #8428
Conversation
|
💚 CLA has been signed |
|
👋 Since this is a community submitted pull request, a Buildkite build has not been started automatically. Would an Elastic organization member please verify the contents of this pull request and kick off a build manually? |
|
Hey @ChrisJamesC 👋🏻 what's the status of this PR? Do you need any guidance? 😄 Let me know and thank you for contributing! |
|
Hi @weronikaolejniczak, thanks for reaching out. |
|
@ChrisJamesC here are some things we'd need to do:
You can strikethrough any other checklist element. If this PR is coming out of a feature request, let's mention it in the description (preferably with closing keywords). Let me know once you've done the changes and I'll test. And if you need any help, don't hesitate to reach out 😄 |
|
Hey @ChrisJamesC 👋🏻 let me know if you need any guidance on moving this PR forward! |
|
👋 Hey there. This PR hasn't had any activity for 90 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment. |
|
❌ We're automatically closing this PR due to lack of activity. Please comment if you feel this was done in error. |
Summary
Propagate the clear prop from
EuiFormControlLayoutDelimitedtoEuiDatePickerRangeso we can clear a date range in a consistent way as for other form elements.QA
Remove or strikethrough items that do not apply to your PR.
General checklist
@defaultif default values are missing) and playground toggles