BowlerStudio icon indicating copy to clipboard operation
BowlerStudio copied to clipboard

Support editing vitamin property names

Open Octogonapus opened this issue 4 years ago • 4 comments

If I create a vitamin property and then want to change its name while still in the new vitamin dialog, I should be able to change it.

Octogonapus avatar Jan 23 '20 04:01 Octogonapus

This is a good idea, but is complex to implement. Once a key is defined, it is added to the cad generator, so changing it is only possible for new sizes of a new vitamin. This is an edge case and no common, and very dangerous since it can render old code non-functional. It may be best to not do this.

madhephaestus avatar Jan 27 '20 18:01 madhephaestus

Once a key is defined, it is added to the cad generator

Don't modify anything external like that until I am done creating the vitamin.

Octogonapus avatar Jan 27 '20 19:01 Octogonapus

sure, but having those fields editable should only happen for the brand new one, not for modifying existing types. You can add new fields, and it will add that field to all instances, but editing might conflict with the code. This is only valid for one of the many use cases and so its stateful and a bit dangerous, and also quite a bit of effort for this narrow edge case.

madhephaestus avatar Jan 27 '20 19:01 madhephaestus

Any time I add a field, that field should not actually be added until I confirm my changes. Do you disagree?

Octogonapus avatar Jan 27 '20 21:01 Octogonapus