-
Notifications
You must be signed in to change notification settings - Fork 54
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
Update: "Wrap" argument documentation requires updating #730
Comments
@validkeys we do not support the I don't think we should need all of this to keep the picker open. I know flatpickr constantly changes how they trigger things switching from |
@rwwagner90 yeah I know it’s not supported, just meant that my previous update to the documentation around stopping the propagation of the events no longer works. So maybe we just want to move it all together since it’s dependent on the inner workings of flatpickr |
It would be good to provide some solutions if we can, I think. |
So since my last PR with regards to controlling the visible
status of the pickr by using an external control, something has changed
and simply stopping the propagation of the click event no longer keeps
the pickr library from closing the popup.
Upon further inspection, pickr has a config option "ignoredFocusElements" that,
given an array of DOM elements, will not perform an internal toggling of the calendar popup
if the external element clicked is in the list of ignore elements.
So here's my current workaround:
The "registerIgnoreElement" adds my external elements to flatpickr's list
of external elements to ignore when it's click outside handler is hit.
I'll try to make a PR when I have some time
The text was updated successfully, but these errors were encountered: