alRex-U / ParCool

A Minecraft Forge Mod for action of Cool Parkour
GNU General Public License v3.0
42 stars 30 forks source link

[1.16.5 Compat Bug] Using Controllable to crawl causes walking speed instead of crawling speed #135

Open Kaleidio opened 1 year ago

Kaleidio commented 1 year ago

Summary Issue is in the title. Does not have the same issue with M/KB

Running Environment

Description This issue is arising because controllable spams inputs that are held, rather than having an available held state to read from. Consider implementing a custom flag check instead of relying on the input classes isHeld method.

More than likely present on all available versions

Kaleidio commented 1 year ago

Issue serves as extension to https://github.com/alRex-U/ParCool/issues/129 and is not a duplicate

alRex-U commented 1 year ago

Thanks for reporting.

Probably I should make "controller" tag for issues

alRex-U commented 1 year ago

Now this is one of the most hardest obstacle to release Controllable support. Even if integrate crawl keybind and controller button as KeyBinding the bahavior are different depending which key pressed.

Internally actually ClientPlayerEntity.isMovingSlowly() returns true in both case, in fact, but only the input from keyboard works correctly.

In addition Controllable should have been implemented like it activate right, left, up, down keybindings when players knock sticks of a controller. Roll not work