whensonZWS / Trigger-Analyzer

2 stars 2 forks source link

Enabling tree layout ignores physics option #6

Open MikillRosen opened 3 years ago

MikillRosen commented 3 years ago

I turn off physics and turn on tree layout but the triggers and stuff move anyway

whensonZWS commented 3 years ago

Changing layout re-enabled the physics in the render engine by default, but this is not reflected in the GUI. (the checked box will not checked/unchecked itself to reflect these). Checked and unchecked the checkbox and the nodes in tree layout will stop. Same behavior happens when loading a map, i.e., the newly loaded map will have physics enabled and in the default (force-directed) layout, even if the checkbox shows otherwise. You need to refresh checked box to update it.

Re-enabled physics upon re-rendering is generally a good thing as the physics is what make the graph-visualization looks decent, so I am hesitant to completely change this behavior.

One solution is to update the GUI (checked box unchecked/checked by themselves) to reflect that the options have been reset. The other one is to respect the current state of option, and let user to re-enabled the physics if the user feels like it.