Reordering node under root

[Giatomo]

I got some issues while moving nodes right under the root node. Mostly the concat function not working here (on jstreer.js at $el.on("move_node.jstree", function(e, data)) :

var oldPath = oldInstance
              .get_path(data.old_parent)
              .concat(nodeText);

Exemple tree :

# (root node)
- A (child node)
- B
- C

I figured out a solution that would improve the consistancy of the jsTreeMoved event of the "from" and the "to" values while also retrieve the positions of the nodes (which is important in my use case but might also be helpful to other users) :
on $el.on("move_node.jstree", function(e, data) :

Shiny.setInputValue("jsTreeMoved:jsTreeR.copied", {
              from: { instance: oldInstanceId, path: oldPath },
              to: { instance: newInstanceId, path: newPath }
            }, {priority: "event"});

can be replaced by either :

Shiny.setInputValue("jsTreeMoved:jsTreeR.copied",{
              from: {
                instance: oldInstanceId,
                parent: data.old_parent,
                path: oldInstance.get_path(node),
                position: data.old_position},
              to: { 
                instance: newInstanceId,
                parent: data.parent,
                path: newInstance.get_path(node),
                position: data.position}
            }, {priority: "event"});

or :

Shiny.setInputValue("jsTreeMoved:jsTreeR.copied"{
              from: {
                instance: oldInstanceId,
                path: [data.old_parent].concat(oldInstance.get_path(node)),
                position: data.old_position},
              to: { 
                instance: newInstanceId,
                path: [data.parent].concat(newInstance.get_path(node)),
                position: data.position}
            }, {priority: "event"});

Thank for looking into this issue !
Best regards

[stla]

Hello,

I vaguely remember I already encountered a similar issue. The code with .concat does not work because the path is not an array here, it is null. Right?

Are you sure that [data.parent].concat will work in all situations? Anyway I'll have to check. These Shiny values are used in the "tree navigator" provided by the package, I'll have to check that your proposal does not break it.

[RightChain]

I'm running into a similar problem. I am using a design that only shows the info in the current directory. Clicks on a directory will navigate to the selected one, while clicks on a file will prime it for being loaded.

All of my files are technically at root level in the tree, and I am getting the following error in my browser console when trying to run a script to detect node movements:

The script looks like this:

jsTreeR_moveInfo_js <- function(ns,id) {
  return(tags$script(paste0('
      // Find the tree element by ID (jsTreeR uses treeID)
      $("#',ns(id),'").on("move_node.jstree", function (e, data) {
        // "data.node.id" is the child being moved
        // "data.parent" is the new parent
        var moveInfo = {
          child: data.node.id,
          parent: data.parent
        };
        // Send to Shiny input$moved_node
        Shiny.setInputValue("',ns('moved_node'),'", moveInfo);
      });
  ')))
}

And the error is:

Uncaught TypeError: oldInstance.get_path(...).concat is not a function
    at HTMLDivElement.<anonymous> (jstreer.js:336:16)
    at HTMLDivElement.dispatch (jquery.js:5145:27)
    at HTMLDivElement.<anonymous> (jquery.js:4949:28)
    at Object.trigger (jquery.js:8629:12)
    at ce.triggerHandler (jquery.js:8713:24)
    at E.jstree.plugins.types.trigger (<anonymous>:3:12642)
    at E.jstree.plugins.types.move_node (<anonymous>:3:63005)
    at HTMLDocument.<anonymous> (<anonymous>:3:112229)
    at HTMLDocument.dispatch (jquery.js:5145:27)
    at HTMLDocument.<anonymous> (jquery.js:4949:28)