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 _onRelease binding in onDragOver handler #62

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

lemming
Copy link
Contributor

@lemming lemming commented May 11, 2022

Easy way to trigger an error is to add draggable attribute to the button in example and try to drag it. There will be a bunch of "Uncaught TypeError: Cannot read properties of undefined (reading 'state')" errors in the console

Screenshot from 2022-05-11 16-17-27

Easy way to trigger an error is to add `draggable` attribute to the button in `example` and try to drag it. There will be a bunch of "Uncaught TypeError: Cannot read properties of undefined (reading 'state')" errors in the console
@nhunzaker
Copy link
Contributor

@cwmanning sorry to be a thorn, but this one looks pretty good too - is it possible to ship this as well?

@sibartlett
Copy link

@cwmanning can we get this merged and released? We're hitting this bug at my company.

@cwmanning cwmanning merged commit 13ab962 into vigetlabs:master Sep 2, 2022
@cwmanning
Copy link
Member

Sure, sorry for the delay! Published as 6.5.2.

@sibartlett
Copy link

Thank you for the quick turnaround today, it's really appreciated 👍🏻

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.

4 participants