Skip to content

Fix/11167: Revamped input handling Android fix - #11647

Merged
kevlahnota merged 6 commits into
Card-Forge:masterfrom
BramTeurlings:feature/archipelago/revamped-input-system
Aug 21, 2026
Merged

Fix/11167: Revamped input handling Android fix#11647
kevlahnota merged 6 commits into
Card-Forge:masterfrom
BramTeurlings:feature/archipelago/revamped-input-system

Conversation

@BramTeurlings

@BramTeurlings BramTeurlings commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Split off from: Archipelago PR
Builds further on the groundwork laid in this PR
Fixes an issue found in this PR which was reverted.

An issue was found with my previous PR (linked above) that was causing players not to be able to move correctly, enter POIs or enter battles on Android. At the time, I was not able to build the Android version of the app locally so I couldn't test the platform, that's now been resolved and I was able to reproduce the observed issue as well as fix it.

It turns out Android movement input is handled by a different mechanism compared to desktops, namely the touchPad and more accurately its knob member.

In this PR I have fixed the issues described above as well as re-staged the commits reverted earlier that improve input handling. Only the changes made from 22327ec on out should be reviewed as everything before that has already been looked at in the abovementioned PR.

Android input bugfix

  • Added a touchKnobInput input vector to GameStage.java
  • Changed the GameHUD.java's touchpad listener so it reports player input to the new touchKnobInput vector in GameStage.java
  • Added touchKnobInput to the input priority ranking created in the reverted PR above such that the priority is: mouse/touch > controller > touchKnob > keyboard.

Everything works for the player animations are jittery on controller now.
haven't tested mobile either but mouse works.

(cherry picked from commit f28811a)
Still not tested on mobile but will do later. (Should be the same as mouse input)

(cherry picked from commit b260851)
(cherry picked from commit 467ab52)
(cherry picked from commit 82714d1)
Added helper function to KeyBinding.java for getting pressed key from libGDX

(cherry picked from commit a631d4f)
@Jetz72 Jetz72 added Adventure For the Adventure Module Android labels Aug 20, 2026
@Jetz72
Jetz72 requested review from kevlahnota and tehdiplomat and removed request for tehdiplomat August 20, 2026 14:33

@kevlahnota kevlahnota left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems to work fine ty

@kevlahnota
kevlahnota merged commit d9146ad into Card-Forge:master Aug 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Adventure For the Adventure Module Android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants