Issue 447: Reload causes extra <span>'s to appear
Status:  WontFix
Owner: ----
Closed:  May 2014
Project Member Reported by wbgil...@gmail.com, Jul 26, 2013
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.


Dynatree - Move Issue - Before.png
4.1 KB   View   Download
Dynatree - Move Issue - After.png
4.0 KB   View   Download
Dynatree - Move Issue - After - Firebug HTML.png
16.7 KB   View   Download
Jul 27, 2013
Project Member #1 moo...@wwwendt.de
could you attach the sample html / js that reproduces this behavior?
Status: Waiting
Labels: Milestone-Release1.2.5
Jul 31, 2013
Project Member #3 wbgil...@gmail.com
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.  :-)

Aug 1, 2013
Project Member #4 moo...@wwwendt.de
>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.

Aug 1, 2013
Project Member #5 wbgil...@gmail.com
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.   
Aug 1, 2013
Project Member #6 moo...@wwwendt.de
Thanks for reporting anyway.
Status: New
Labels: -Milestone-Release1.2.5 Milestone-Release2.0
May 1, 2014
Project Member #7 moo...@wwwendt.de
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
Status: WontFix