Move effect callback to the maxifill

We need a call when the Animation's effect/target changes but that's future work when we implement mutation.
We should have a test to make sure we're called with null after the player is cancelled.