EntityCommandMovement feels weird

[valkyrienyanko]

I feel like EntityCommandMovement should not be a thing, it should rather be merged into Entity. But then this takes away the flexibility of having "non-moving" entities. I think we should assume that all entities will move and worry about non-moving entities later on.

I don't like that the movement properties are in 2 different places. I found it very confusing as I forgot that they were split up.

[valkyrienyanko]

But then this takes away the flexibility of having "non-moving" entities.

Moving entities should extend from CharacterBody2D as they do now. Non-moving entities should extend from StaticBody2D. So non moving entities should not be implemented in Entity at all.

Perhaps Entity and all EntityCommands should be renamed to MovingEntity / MovingEntityCommand[...]

And non moving entity would just be called Entity.