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

[Fix]: When destroying, failed to remove the dragover and dragenter events. #2373

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

Conversation

KonghaYao
Copy link

When I repeated the destroy and mount events, I found that the dragover and dragenter events mounted by Sortablejs were not destroyed, which caused a bug. So I checked the source code and fixed the issue.

@KonghaYao
Copy link
Author

image
It looks like the tests on circleCI failed, but they were successful on my device. Windows 10.

@KonghaYao
Copy link
Author

Because there was an error in the CircleCI pipeline, I upgraded testcafe and its related configurations directly so that it can run test scripts for everyone.
image

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.

1 participant