search

Seneral / Node_Editor_Framework

A flexible and modular Node Editor Framework for creating node based displays and editors in Unity
https://nodeeditor.seneral.dev
MIT License
2.01k stars 414 forks source link

Canvas file reference lost when Loading/Saving #63

Closed Salepate closed 7 years ago

Salepate commented 8 years ago

Hi,

Whenever Loading / Saving a canvas, there is that backup mechanic which create the LastSession.asset. This asset creation leads to losing the reference to the original asset for the currently edited canvas. Although its not a problem when saving. It displays the wrong information inside the Editor

image

Might be confusing for the end-user when editing multiple canvas.

Best Regards

Seneral commented 8 years ago

Yep that was only caused in the last rewrite of the cache system and planned to fix this but yet did not found time for it... The problem is that the canvas name is always the name of the saved file. We could save it with it's original name so we would have to store additional information about the cache. I can take a look how to fix this:)

Salepate commented 8 years ago

I temporarily cached the name to bypass that side effect indeed.

keep up the good work

Seneral commented 8 years ago

Cool! Maybe you can create a pull request? :)

Seneral commented 8 years ago

Planning on restoring some parts of the Cache System we had long ago, which basically saved the last canvas with it's name and stored the name in a string in the registry through editorprefs. It did work but when I changed it to lastsession.asset I did not realize at that point it would replace the name:/

Seneral commented 7 years ago

Has been fixed in 91bacfc :)