Double jump bug about sample-platformer HOT 3 CLOSED

@apaatsio Good find! Double jumps definitely not intentional.

Must be something odd about https://github.com/excaliburjs/sample-platformer/blob/master/src/bot.ts#L59 allow the jump to be registered https://github.com/excaliburjs/sample-platformer/blob/master/src/bot.ts#L99

from sample-platformer.

Okay, this is a subtle issue but I believe this is because the collision resolution runs after actor update.

Immediately after the jump the actor update is complete and the jump impulse won't be applied to position until the next frame, so still colliding.

In the same frame, collision is processed and the actor geometry is still intersecting so onGround is set back to true, making a double jump possible in the very next frame.

Potential solutions:

• Switch to onPreUpdate to ensure impulse is processed the same frame as collisions
• Shift the actor pos.y up to keep from re-colliding with the floor the same frame as the jump

from sample-platformer.

@apaatsio I switched to onPreUpdate seems to do the right thing, thanks for the find!

from sample-platformer.