Skip to content

Fixing issue with DragEnd#119

Open
k2tomasz wants to merge 1 commit intoPostlagerkarte:masterfrom
k2tomasz:master
Open

Fixing issue with DragEnd#119
k2tomasz wants to merge 1 commit intoPostlagerkarte:masterfrom
k2tomasz:master

Conversation

@k2tomasz
Copy link
Copy Markdown

No description provided.

@ViRuSTriNiTy
Copy link
Copy Markdown
Collaborator

Could you please create an issue and describe the bug or at least add a description to the PR? The link between your comment in PR #97 and this PR is hard to find / understand. I spent like 30 minutes to figure out what was going on here ^^.

Also I think we need to discuss the fix first as the added conditions now assume that OnDragEnd is always called after OnDrop - which is not the case in my application for example. When I add your change, I get an DragDropService that is not correctly reset.

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.

2 participants