Is this complicated to understand? In my tool every change is shown in isolation as a Step in the review process.
green with + = new node
green outline = new node, but it is not the focus in the current review step
orange with ~ = has changes
orange outline = has changes, but it is not the focus in the current revuew step
red with - = removed node
red outline = removed node, but it is not the focus in the current review step
OR a single Visual review step at the end with everything
green with + = new node
orange with ~ = has changes
red with - = removed node
These +~- buttons are clickable. When you click on them it redirects you to the property changes step
1 Like
Looks really nice, good wook 
I would suggest refining these changes into:
- visual changes only - the node only moved or was added to a group e.g. changes to 'x', 'y', 'w', 'h', 'g' attributes
- visual-semantic - logically changes made visually, e.g. changes to "wires", "d", "nodes" attributes. Node was disabled or a group was added to or a new wire output wire was defined on the node.
- logical/textual changes - something changed that affects the logic or workings of the entire flow, e.g. all other node attributes (unless this is applied to specifically to specific nodes)
The intention is to make reviewing faster since changes that are purely visual can basically be ignored since their have no consequences to the workings of the flow.
Visual-semantic changes can be solely compared visually, i.e. using the visual diff. Logically/textual changes require comparing the internals of the nodes and a visual-only comparison won't be enough and some kind of textual comparison will be necessary.
1 Like
