theia-ide / sprotty

A next-gen web-based graphics framework
Apache License 2.0
138 stars 23 forks source link

Removing moveModule breaks auto-layout #225

Open JanKoehnlein opened 6 years ago

JanKoehnlein commented 6 years ago

For static diagrams, in which nodes cannot be moved, I expected removing the moveModule from the diagram config would do the trick. But this breaks the animation of auto-layout and diagram updates to an extend that the result is unusable.

The right way to do this is to remove the moveFeature from the respective nodes.

spoenemann commented 6 years ago

Removing modules often has unexpected side effects. We should rethink the whole approach of having multiple modules.