Topic: Mistake in Tree demo files

Hi moro,

Here's what I found. In "Tree row highlighting" demo, row that says "drag me! sorry imposible" *is* in fact draggable.

You have to fix the following file:

C:\xampp\htdocs\MifTree.SVN\assets\files\forest.json, from line 9 should be updated as:

        "property": {
            "name": "drag me! sorry imposible",
            "dragDisabled" : true,    // This is missing.
            "hasCheckbox": false
        },

Also, after updating to latest version I did thorough code comparing and here is the part I don't understand.

In Plugins\Mif.Tree.Drag.js, line 71 in mousedown() method:

    mousedown: function(event){
                ...
        if(this.current.dragDisabled){
            this.current=false;
        }
                ...
    },

In earlier versions you had current.DDisabled, but now it says current.dragDisabled.

That was necessary change, since you did it. But could you please tell me if "dragDisabled" refers to a node property in JSON file or is it simply a private variable?

Have you changed globally DDisabled to dragDisabled?

Thanks again for excellent work.

Temuri

Thumbs up Thumbs down

Re: Mistake in Tree demo files

temuri wrote:

Have you changed globally DDisabled to dragDisabled?

yes, it's right name for this property. see docs: dragDisabled - (boolean) disable dragging

p.s. tree.drag will be rewriten soon for support d'n'd between trees .