idanarye / bevy-tnua

A floating character controller for Bevy
https://crates.io/crates/bevy-tnua
Apache License 2.0
235 stars 16 forks source link

Merge TnuaPlatformerAnimatingOutput's functionality into TnuaPlatformerState #26

Closed idanarye closed 11 months ago

idanarye commented 1 year ago

My current leading design in #24 would allow the movement actions to have a state. Users will want to animate the actions, and I can have each action have its own animating output type with information for controlling the animation for that action, but having two types for each action starts to seem like a bit too much. I'm considering, instead, to use the action's state for both state and animation.

This got me thinking - maybe the same should be done for regular animating output? My reasoning is:

Note that TnuaManualTurningOutput will still be a separate component, because it affects the behavior.

idanarye commented 11 months ago

This is actually already in. Well, not really, as TnuaPlatformerState is gone - but the TnuaController which replaces it actually merges the controls, the state, and the animating output.