Skip to content

feat(input): movement-first facing with conditional aim override#154

Merged
f1cklepickle merged 3 commits intomainfrom
feat/input-movement-first-facing-conditional-aim-override
Mar 8, 2026
Merged

feat(input): movement-first facing with conditional aim override#154
f1cklepickle merged 3 commits intomainfrom
feat/input-movement-first-facing-conditional-aim-override

Conversation

@f1cklepickle
Copy link
Copy Markdown
Owner

@f1cklepickle f1cklepickle commented Mar 7, 2026 •
edited
Loading

Why

Current always-aim-facing behavior felt disconnected from movement. This change makes movement the default facing source and only enables aim-facing during explicit combat-aim intent.

What changed

  • Added PlayerFacingPolicyResolver to choose facing source from movement vs aim intent
  • Updated PlayerController to delegate facing selection and keep controller orchestration thin
  • Wired facing policy on Player.prefab
  • Added EditMode and PlayMode tests for policy contracts, source switching, hysteresis, and fallback behavior
  • Added mobile tuning control for right-stick attack deadzone and tests for propagation/clamping
  • Updated Docs/TEST_LOG.md with feat(input): movement-first facing with conditional aim override #153 validation entry

How to test

  1. Open the relevant scene or demo (if applicable)
  2. Press Play → verify expected behavior
  3. Run tests:
    • EditMode
    • PlayMode (or note manual validation)

Checklist

  • Unit tests (EditMode) added or updated
  • PlayMode test or manual validation included
  • Demo scene updated (if player-visible)
  • Prefab links, tags, and layers validated
  • README / Docs touched

Related

@f1cklepickle f1cklepickle added the apk Opt-in signal to generate an Android APK build for testing (CI artifacts). label Mar 7, 2026
@f1cklepickle f1cklepickle changed the title Feat/input movement first facing conditional aim override feat(input): movement-first facing with conditional aim override Mar 8, 2026
@f1cklepickle f1cklepickle added feature New feature or request testing Work specifically related to adding or fixing test setups gameplay Anything affecting the core loop (combat, waves, health). and removed apk Opt-in signal to generate an Android APK build for testing (CI artifacts). labels Mar 8, 2026
@f1cklepickle f1cklepickle added scene Scene updates, environment layout, prefab arrangement. prefab Adding/updating prefabs. labels Mar 8, 2026
@f1cklepickle f1cklepickle merged commit a64727a into main Mar 8, 2026
3 checks passed
@f1cklepickle f1cklepickle deleted the feat/input-movement-first-facing-conditional-aim-override branch March 8, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature New feature or request gameplay Anything affecting the core loop (combat, waves, health). prefab Adding/updating prefabs. scene Scene updates, environment layout, prefab arrangement. testing Work specifically related to adding or fixing test setups

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat(input): movement-first facing with conditional aim override

1 participant

@f1cklepickle