[BUG] Review system update issue

[acardona]

When deleting a node, the review widget will refresh the tables, without considering whether the revision was for the whole arbour or just for a subarbor. So one starts reviewing a subarbor, deletes a node, and the review tables change completely to include the whole arbour.

[acardona]

Also, closing the Review widget throws an error.

[acardona]

And once closed it can't be reopened again. The javascript console throws errors: "cannot read properties of undefined (reading 'EVENT_NODE_DELETED'), in the ReviewSystem.destroy() method.

[acardona]

Furthermore, upon deleting a skeleton treenode, the view is repositioned to the last node of the segment (the first to review). Frankly the view should not move at all, it gets in the way a lot.

[tomka]

And once closed it can't be reopened again. The javascript console throws errors: "cannot read properties of undefined (reading 'EVENT_NODE_DELETED'), in the ReviewSystem.destroy() method.

This is fixed now in dev.

[tomka]

So one starts reviewing a subarbor, deletes a node, and the review tables change completely to include the whole arbour.

This is fixed in dev as well. The review table is now updated so that it respects any active sub-arbor review.

Furthermore, upon deleting a skeleton treenode, the view is repositioned to the last node of the segment (the first to review). Frankly the view should not move at all, it gets in the way a lot.

The view is now also not repositioned anymore after a node deletion.