[Bug][Beta] Fix "Critical Hit" message appearing before dealing damage

[innerthunder]

What are the changes the user will see?

The message "A critical hit!" now correctly appears after damage is dealt.

Why am I making these changes?

fixes #4251

What are the changes from a developer perspective?

field/pokemon: The critical hit message generated in Pokemon#apply now occurs after the call to Pokemon.damageAndUpdate instead of before.

Screenshots/Videos

PokeRogue.-.Google.Chrome.2024-09-14.11-04-56.mp4

How to test the changes?

1. Give your Pokemon Flower Trick with MOVESET_OVERRIDE in src/overrides.ts
2. Use Flower Trick
3. The critical hit message should happen after damage animations

Checklist

• I'm using beta as my base branch
• There is no overlap with another PR?
• The PR is self-contained and cannot be split into smaller PRs?
• Have I provided a clear explanation of the changes?
• Have I considered writing automated tests for the issue?
• If I have text, did I make it translatable and add a key in the English locale file(s)?
• Have I tested the changes (manually)?
  • Are all unit tests still passing? (npm run test)
• Are the changes visual?
  • Have I provided screenshots/videos of the changes?