search

Franklinjagan / jsplumb

Automatically exported from code.google.com/p/jsplumb
0 stars 0 forks source link

removeAllEndpoints and removeEndpoint fail when a container is specified #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Me again, experimenting container stuff :)

I found out that the 2 mentioned functions forget to pass the Endpoint 
container when they call _removeElement, resulting in undead Endpoints.

Adding the container as the second argument of _removeElement fixed this issue.

Original issue reported on code.google.com by sylvain....@gmail.com on 23 Sep 2010 at 9:05

GoogleCodeExporter commented 9 years ago 
Found another one: in the Endpoint drag stop event callback, line 1144, the 
container is not given to _removeElements, adding it cleans up the floating 
endpoint and the drag helper correctly.

Original comment by sylvain....@gmail.com on 23 Sep 2010 at 9:16

GoogleCodeExporter commented 9 years ago 
these are all fixed now too.

Original comment by simon.po...@gmail.com on 24 Sep 2010 at 4:47

GoogleCodeExporter commented 9 years ago 
fixed in 1.2.3

Original comment by simon.po...@gmail.com on 9 Oct 2010 at 11:51

GoogleCodeExporter commented 9 years ago

Original comment by simon.po...@gmail.com on 27 Nov 2010 at 1:18