NodeEditor: (re)setting the nodes prop does not update the editor contents

[chrisjpatty]

NodeEditor: (re)setting the nodes prop does not update the editor contents

[NicWickman]

Signal boosting. This is a must.

[chrisjpatty]

Ah, yeah you've discovered a constraint here with how the editor is currently designed. Right now the editor goes through a reconciliation phase if the editor is loaded with existing nodes. That reconciliation phases goes through the supplied nodes and coerces them to match the schema of the node types and input types. It could potentially become a performance hit if the editor needed to re-run the reconciliation every time the provided nodes change from the parent component. For that reason and a few others, the node editor technically always runs in an "uncontrolled" way, and internally manages its own state. But you're right that this can lead to some unexpected behavior when you try to manually provide a new set of nodes.

Would it work for you if there was an imperative function exposed on the editor that would force the reconciliation to run and reset the the nodes? Something like:

const nodeEditor = React.useRef();

const setNewNodes = () => {
  nodeEditor.setNodes({...newSetOfNodes})
}

return (
  <div>
    <button onClick={setNewNodes}>Set New Nodes</button>
    <NodeEditor ref={nodeEditor} />
  </div>
)

There may be a better way to handle this in the future to with some more advanced reconciliation of previous and current nodes, but would this work for your use case in the meantime?

[andraz-at]

Yes, if I can get NodeEditor ref function to trigger an update manually, then I can control the updates that way.

p.s.: The connections bug was on the store side, NodeEditor now renders all connections ok, example on link updated.

[netgfx]

Is this a feature are are waiting on? or is there something we can use to dynamically add nodes?