Due to race conditions (editor tries to center the corresponding part which might not have come into view on deserialization), we don't preserve editor prop state. I had added this to Part.serialize() but forgot to do so for Image and WorldStack.
This was a bit of a pain to debug b/c deserializing the halo was open not the editor: the editor-open prop on Image was causing halo-open to be set to true, but the editor itself was not open. Potentially we want to decouple halo and editor but visually it does make sense to have the former open when the latter is...
Main Points
Due to race conditions (editor tries to center the corresponding part which might not have come into view on deserialization), we don't preserve editor prop state. I had added this to Part.serialize() but forgot to do so for Image and WorldStack.
This was a bit of a pain to debug b/c deserializing the halo was open not the editor: the
editor-openprop on Image was causinghalo-opento be set to true, but the editor itself was not open. Potentially we want to decouple halo and editor but visually it does make sense to have the former open when the latter is...