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

feat(datepicker): add onOpen callback #8705

Merged

Conversation

tay1orjones
Copy link
Member

Closes #8698

This adds an onOpen callback prop that exposes flatpickr's onOpen to consumers while still calling the internal onHook.

Changelog

New

  • datepicker: add onOpen callback

Changed

  • datepicker-story: add onClose and onOpen callback props as actions

Testing / Reviewing

Datepicker should function the exact same as it did before with no regressions. The playground story should show action functions being called on open and close of the datepicker passing along the expected args.

@netlify
Copy link

netlify bot commented May 18, 2021

Deploy Preview for carbon-elements ready!

Built with commit a4ce010

https://deploy-preview-8705--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented May 18, 2021

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables with commit a4ce010

https://deploy-preview-8705--carbon-components-react.netlify.app

@tay1orjones tay1orjones requested a review from a team as a code owner May 18, 2021 14:29
Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@andreancardona
Copy link
Contributor

@tay1orjones any clue what's going on with the CI checks?

@tay1orjones
Copy link
Member Author

@andreancardona There's an ongoing incident with GitHub actions. It sounds like it should be coming back up soon

@tay1orjones tay1orjones mentioned this pull request May 18, 2021
87 tasks
@kodiakhq kodiakhq bot merged commit 849c8c1 into carbon-design-system:main May 18, 2021
@jnm2377 jnm2377 mentioned this pull request May 24, 2021
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add onOpen event handler to DatePicker
4 participants