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

Add handleInteractionEnd to dropFromOutside to cleanup preview. #2690

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ardenc
Copy link

@ardenc ardenc commented Dec 3, 2024

Howdy,

I as well as a few other users have had a problem with trying to manually control the state for items dropped from outside.

Current behavior drops the preview onto the calendar. This is not desired as it interferes with trying to adjust the date and event after drop.

Use Case Example: Not allowing dropping from outside on specific calendar days.

Related issues
#2675
#2294

If you want me to change anything just let me know!

@proohit
Copy link

proohit commented Dec 7, 2024

Hey, I just tried this patch but it still drops the preview event onto the calendar. How did you get it to work? I want the preview still to be visible while dragging and then update my state manually with onDropFromOutside

EDIT: nevermind, I forgot to actually update the onEventDrop to update my events. So yeah, thanks for this PR!

@ardenc
Copy link
Author

ardenc commented Dec 17, 2024

@cutterbl Hi, sorry to ping you. Would you be able to review this pull request to see if it is something that can be incorporated into the source?

If you have any questions or want me to update anything just let me know.

,Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants