minExpandLevel does not auto-expand

[SpencerTM]

I am trying to migrate from dynatree to fancytree.

I have a tree that is loaded with json.

Some settings are

checkbox: true,
selectMode: 3,
icons: false,

When I set minExpandLevel:2 only the root node shows and the ability to expand it is gone.

When I remove the minExpandLevel the tree loads and can be expanded manually.

I set the debugLevel: 2 and $.ui.fancytree.debugInfo = 2; but nothing shows in the console.

Sample html & json files are attached to the question in google groups:
https://groups.google.com/forum/#!topic/fancytree/Ve3SJF86DOY

This still happens with fancytree 2.3.0.

[mar10]

The desired behavior for minExpandLevel > 1 is (for nodes with level <= minExpandLevel):

• the node should always be expanded
• expand/collapse on dblclick is disabled
• the expander icon should be removed (top level nodes only)

Dynatree behaves like this:http://wwwendt.de/tech/dynatree/doc/sample-minexpand.html

In Fancytree currently only the expander is removed.

(One workaround is to explicilty set 'expanded' on toplevel nodes)

[flack]

@mar10 I just tried your fix on the latest release, but AFAICT it does not cover all cases: I use minExpandLevel: 2, and no other option on a <ul></li> tree, and I still get the same problem as reported in this ticket.

I really don't know anything about the fancytree codebase, but looking at your patch, it seems to me that these three lines here

fancytree/src/jquery.fancytree.js

Line 2853 in 70efd20

if( !node.lazy ) {

are wrongly placed. The if(level > 1){ check in the line above means that only nodes from the second level downwards will get extended. If I read the code correctly, that is :-)