-
Notifications
You must be signed in to change notification settings - Fork 240
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
Removing items requires a bounding box #95
Comments
After experimenting a little, it seems that remove doesn't work without a matching bbox. |
We experienced the same issue when removing, updating, and re-inserting the same item quickly. We looked at the code and the issue is that One thing that got us confused at first is why there needs to be a spatial search prior to removing an element given that the docs say elements are removed by reference. Would it be possible/make sense to have a |
id
doesn't work
Actually probably the simplest way would be to keep a reference to parent for all nodes and iitems. This also helps greatly with item updates and reinsertions. |
@thisredone thank you for this fork, this is exactly what I need right now! Testing it immediately :) |
Change the code |
I know the callback is never called because I never see the log statement.
The text was updated successfully, but these errors were encountered: