What I Did Do:

Made it so that clicking on "Graph Editor" while in a saved model causes the graph information to de-render and a prompt asking for save to render in its place.

Made it so that regardless of answer to prompt, the newly rendered graph information is blank.

Moved the save node logic up into index.js out of graph-builder.js to allow for rendering of previously specified prompt.

What I Did Not Do:

Clicking "Graph Editor" while in a model that hasn't been saved yet does not cause the graph information to reset or a save prompt to appear. This is because the logic tied to the rendering of the save prompt is dependent on props changing, and the already existing save node logic depends on the model being saved in the first place.

Recommendation

Create a new button on the Graph Editor called "Reset" that uses the function "clearPrompt()" that I defined in index.js. It can basically be stuck anywhere on the page of your choosing and if you decide you want it and know where you want it it's not big trouble for me to just do this myself.

Clicking Model List does not prompt for save. This is because at the moment the highest level of parent child relationship supporting the save functionality is index.js inside the graph-editor folder, which basically means that save functionality is tied to the graph editor itself, so routing away from the graph editor routes away from save functionality.

Recommendation

The way it's implemented now I don't think it's possible to implement this functionality without moving the save logic again. Either make a higher level component that parents both the graph list and the graph editor, or maybe there's something you can do where saving is put on its own route. This is something in general though that I don't really want to continue working on.

OrderAndCh4oS / Graph-Editor

Regarding Issue #17 #19

What I Did Do:

What I Did Not Do:

Recommendation

Recommendation