UnitedLexCorp / SimpleTalk

An implementation of HyperTalk in Ohm-js
Apache License 2.0
7 stars 1 forks source link

Deserialized script editor does not save script #191

Open dkrasner opened 3 years ago

dkrasner commented 3 years ago

Main Points

If you save/serialize a Snapshot with a scripting editor open, it will be there when you load/de-serialize. However, the "save script" button will fail to update the target part's script property.

The reason for this is that the "save script" button does not have a proper script. The handler is defined under the hood and is added when System responds to an openScriptEditor command. The net result is that the button is de-serialized but has no click handler.

We should either add the script there - sorting out the preventing issue which sends some sort of infinite recursion error - to the button or build the entire editor in ST (related to #150)