added some more placed where we flag graph as modified and undo steps

ICRAR · Oct 31, 2024 · d8195dc · d8195dc
1 parent d116ab4
commit d8195dc
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 0 deletions.
diff --git a/src/Eagle.ts b/src/Eagle.ts
@@ -2901,6 +2901,7 @@ export class Eagle {
             this.addEdge(srcNode, srcPort, destNode, destPort, edge.isLoopAware(), edge.isClosesLoop(), () => {
                 this.checkGraph();
                 this.undo().pushSnapshot(this, "Add edge");
+                this.logicalGraph().fileInfo().modified = true;
                 // trigger the diagram to re-draw with the modified edge
                 this.logicalGraph.valueHasMutated();
             });
@@ -2947,6 +2948,7 @@ export class Eagle {
             this.addEdge(srcNode, srcPort, destNode, destPort, edge.isLoopAware(), edge.isClosesLoop(), () => {
                 this.checkGraph();
                 this.undo().pushSnapshot(this, "Edit edge");
+                this.logicalGraph().fileInfo().modified = true;
                 // trigger the diagram to re-draw with the modified edge
                 this.logicalGraph.valueHasMutated();
             });
@@ -3530,6 +3532,7 @@ export class Eagle {
                 this.addEdge(realSourceNode, realSourcePort, realDestNode, realDestPort, false, false, (edge: Edge) => {
                     this.checkGraph();
                     this.undo().pushSnapshot(this, "Add edge " + edge.getId());
+                    this.logicalGraph().fileInfo().modified = true;
                     this.logicalGraph.valueHasMutated();
                 });
 
@@ -3543,6 +3546,7 @@ export class Eagle {
                 this.addEdge(realDestNode, realDestPort, realSourceNode, realSourcePort, false, false, (edge: Edge) => {
                     this.checkGraph();
                     this.undo().pushSnapshot(this, "Add edge " + edge.getId());
+                    this.logicalGraph().fileInfo().modified = true;
                     this.logicalGraph.valueHasMutated();
                 });
 
@@ -4000,6 +4004,7 @@ export class Eagle {
             // refresh the display
             this.checkGraph();
             this.undo().pushSnapshot(this, "Change Node Parent");
+            this.logicalGraph().fileInfo().modified = true;
             this.selectedObjects.valueHasMutated();
             this.logicalGraph.valueHasMutated();
         });
@@ -4048,6 +4053,7 @@ export class Eagle {
             // refresh the display
             this.checkGraph();
             this.undo().pushSnapshot(this, "Change Node Subject");
+            this.logicalGraph().fileInfo().modified = true;
             this.selectedObjects.valueHasMutated();
             this.logicalGraph.valueHasMutated();
         });
@@ -4246,6 +4252,7 @@ export class Eagle {
                 }
 
                 this.checkGraph();
+                this.logicalGraph().fileInfo().modified = true;
                 this.undo().pushSnapshot(this, "Add field");
             });
 
@@ -4724,6 +4731,7 @@ export class Eagle {
         this.flagActiveFileModified();
         this.checkGraph();
         this.undo().pushSnapshot(this, "Edit Node Category");
+        this.logicalGraph().fileInfo().modified = true;
         this.logicalGraph.valueHasMutated();
     }
 

diff --git a/src/GraphRenderer.ts b/src/GraphRenderer.ts
@@ -1144,11 +1144,13 @@ export class GraphRenderer {
                     if (parent !== null && outerMostNode.getParentId() !== parent.getId() && outerMostNode.getId() !== parent.getId() && !ancestorOfParent && !outerMostNode.isEmbedded()){
                         GraphRenderer.updateNodeParent(outerMostNode, parent.getId(),  allowGraphEditing);
                         GraphRenderer.NodeParentRadiusPreDrag = eagle.logicalGraph().findNodeByIdQuiet(parent.getId()).getRadius()
+                        eagle.logicalGraph().fileInfo().modified = true;
                     }
 
                     // if no parent found, update
                     if (parent === null && outerMostNode.getParentId() !== null && !outerMostNode.isEmbedded()){
                         GraphRenderer.updateNodeParent(outerMostNode, null,  allowGraphEditing);
+                        eagle.logicalGraph().fileInfo().modified = true;
                     }
 
                     if (oldParent !== null){
@@ -2211,6 +2213,7 @@ export class GraphRenderer {
         eagle.addEdge(srcNode, srcPort, destNode, destPort, loopAware, closesLoop, (edge : Edge) : void => {
             eagle.checkGraph();
             eagle.undo().pushSnapshot(eagle, "Added edge from " + srcNode.getName() + " to " + destNode.getName());
+            eagle.logicalGraph().fileInfo().modified = true;
             eagle.logicalGraph.valueHasMutated();
             GraphRenderer.clearEdgeVars();
         });

diff --git a/src/LogicalGraph.ts b/src/LogicalGraph.ts
@@ -348,6 +348,7 @@ export class LogicalGraph {
 
     addGraphConfig = (config: GraphConfig): void => {
         this.graphConfigs.push(config);
+        Eagle.getInstance().undo().pushSnapshot(Eagle.getInstance(), "Added a new graph config");
     }
 
     duplicateGraphConfig = (config: GraphConfig): void => {
@@ -367,6 +368,8 @@ export class LogicalGraph {
             setTimeout(() => {
                 $('#graphConfigurationsTableWrapper .activeConfig .column-name input').focus().select()
             }, 100);
+
+        Eagle.getInstance().undo().pushSnapshot(Eagle.getInstance(), "Duplicated a graph config" + clone.getName());
     }
 
     removeGraphConfig = (config: GraphConfig): void => {