dangbroitsdon / Wall-Jump-Unofficial

DEAD
https://www.curseforge.com/minecraft/mc-mods/wall-jump-unofficial-arch
Other
3 stars 5 forks source link

[1.18, 1.19] Slow flying in creative mode and spectator mode #9

Closed artemisSystem closed 2 years ago

artemisSystem commented 2 years ago

I noticed that:

If i comment out the miscellaneousTickMovement mixin in ClientPlayerEntitySpeedBoostMixin, these issues do not occur, so i assume there are some wrong assumptions being made in the SpeedBoostLogic.doSpeedBoost() method. This does not occur in the forge 1.18.2 version of wall jump.

artemisSystem commented 2 years ago

After more research, i've found commenting out this line: https://github.com/dangbroitsdon/Wall-Jump-Unofficial-Fabric/blob/1.19-fabric/src/main/java/genandnic/walljump/client/SpeedBoostLogic.java#L31 fixes it. It looks basically the same as the forge version though (https://github.com/genandnic/Wall-Jump/blob/1.18.1/src/main/java/genandnic/walljump/client/SpeedBoostLogic.java#L27) except it uses a function instead of an assignment. Maybe there are some slight differences to how this works between forge and fabric, somehow? I remember also encountering this in the other unofficial fabric port (https://github.com/padlocks/Wall-Jump/blob/1.18-fabric/src/main/java/genandnic/walljump/mixin/client/ClientPlayerEntitySpeedBoostMixin.java#L47), so it may be something like that.

artemisSystem commented 2 years ago

yup, changing it to pl.airStrafingSpeed =... fixes it. PR incoming

dangbroitsdon commented 2 years ago

thank you for contributing my boy i appreciate it on a whole nother level. changes shall be merged!

artemisSystem commented 2 years ago

Glad you like it. Please don't call me boy though, i'm not a boy 🥴. In general, i would advise against calling random strangers on the internet "my boy".

dangbroitsdon commented 2 years ago

my apologizes, i be calling everyone everything nowadays