MontyTRC89 / Tomb-Editor

Tomb Editor
55 stars 15 forks source link

TE: Simplify Object transform #839

Open Raildex opened 1 month ago

Raildex commented 1 month ago

TombLib provides several Interfaces: ISpatial IRotatableY IRotatableYX IRotatableYXRoll IColorable ISizeable IScalable

These interfaces are implemented or not by subclasses of PositionBasedInstance

Lots of code (namely EditorActions.cs) checks for this interface to act accordingly.

I propose to pull all of that into PositionBasedInstance instead to simplify the code. The code checks for specific Engine version as well, which further complicates things. I suggest to simply throw a warning when a transform is about to happen the engine does not support and ignore it.