search

jannbmsy / dynatree

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

Reload causes extra <span>'s to appear #447

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Load dynatree with initial (level 1) nodes.
2. Drag a node before/after another to reorder the nodes.
3. Dynatree .reload causes two extra span's to be added to the dynatree (see 
attached image)
   Those extra spans are the following HTML markup:

   <span class="dynatree-connector"></span>
   <span class="dynatree-icon"></span>

What is the expected output? What do you see instead?
   Dynatree should render nodes as they were before the drag-and-drop.

Please use labels and text to provide additional information.
   See attached images for before, after and the incorrect HTML after the drop.

Original issue reported on code.google.com by wbgil...@gmail.com on 26 Jul 2013 at 11:30

Attachments:

GoogleCodeExporter commented 8 years ago 
could you attach the sample html / js that reproduces this behavior?

Original comment by moo...@wwwendt.de on 27 Jul 2013 at 7:01

GoogleCodeExporter commented 8 years ago 
[deleted comment]
GoogleCodeExporter commented 8 years ago 
That'd be a lot of code (you've worked w/ our apps/framework before. :-)
But, Firebug shows dynatree's .reload (r609) is called and then this happens.  
I know some javascript, but your stuff is over my head.  :-)

Original comment by wbgil...@gmail.com on 31 Jul 2013 at 9:40

GoogleCodeExporter commented 8 years ago 
>That'd be a lot of code (you've worked w/ our apps/framework before.
Yeah, I remember ;-)

I tried to reproduce with the standard DND samples (I added a button that calls 
tree.reload())
But it looks Ok.
If you can tweak one of the standard samples (e.g. doc/sample-dnd.html) so that 
the problem turns up, that would be a great help.

Original comment by moo...@wwwendt.de on 1 Aug 2013 at 9:32

GoogleCodeExporter commented 8 years ago 
It has something to do with the way nodes are added, but I looked at dynatree 
code and couldn't figure it out.  Dynatree was adding the extra spans for some 
reason.  I'm mid-level w/ javascript, not as advanced as how some of that code 
works w/o studying javascript more.  We're good now, I add them a different way 
and it works like we need it to.

Original comment by wbgil...@gmail.com on 1 Aug 2013 at 3:49

GoogleCodeExporter commented 8 years ago 
Thanks for reporting anyway.

Original comment by moo...@wwwendt.de on 1 Aug 2013 at 5:47

GoogleCodeExporter commented 8 years ago 
As of 2014 Dynatree is feature frozen.
Please have a look at Fancytree (sequel of DynaTree 1.x): chances are good that 
the problem was resolved / the requested featuer is already implemented.
Please open a new issue there otherwise:

https://github.com/mar10/fancytree

Original comment by moo...@wwwendt.de on 1 May 2014 at 4:38