shunjizhan / use-tree-state Goto Github PK

should wrap reducers in useCallback, to avoid unnecessary rerender

reducers.addNode(path: array, hasChildren: bool) is more general

define custom errors to be more consistent

• FindNodeError
• ToggleOpenError
• AddNodeError

instead of knowing the new state, it is also useful to know the previous event.

onChange({ newState, e })

this will be more convenient for users

reducers.renameNodeByProp('name', 'snorlax', 'pikachu')
reducers.deleteNodeByProp('url', '', 'root/children1')
...

It would be nice to see a configuration option that allows you to have only one child node open for any parent node at one time.

• single node
• parent node1 (open)
• • child node1 (closed)
• • child node2 (open)
• • child node3 (close) - If we open this, child node2 closes automatically
• parent node2 (closed)
• parent node3 (closed) - if we open this, parent node1 closes automatically (and all its child nodes)

so from outside use won't change it by accident

now all the reducers takes a path param, but might be useful to be able to do renameNode(oldname: string, newName:string)

internally we can do

renameNodeByName = (name, newName) => {
  const path = findPathByProp('name', name);
  renameNode(path, newName)
}

currently there is no way to reinitialize tree state

const { reinitialize } = useTreeState();
reinitialize({ data, options, customreducers })