Secretchronicles / TSC

An open source two-dimensional platform game.
https://secretchronicles.org/
GNU General Public License v3.0
205 stars 49 forks source link

Analog Jump Broken - Needs to be Removed... #581

Closed datahead8888 closed 8 years ago

datahead8888 commented 8 years ago

Turning analog jump on in the options menu, then pressing up on an analog joystick while in a level, does not make Alex jump.

This doesn't appear to work in version 2.0, either, which probably says a lot about how many people are using this feature...

datahead8888 commented 8 years ago

I have a tentative solution started for this. The original issue was that the jump processing code that checks for analog upward presses was never called in the first place when up is pressed.

If analog jump is turned on in a level and if up is pressed on the analog stick, I am wanting it to:

Not dealing with these situations seems to make it a little unpredictable whether Alex will jump or climb when touching a climbable object (I found different behavior climbing while standing on the ground or trying to climb during a jump before).

Related: #570

Furthermore, analog jumping will not be applied for the directional pad. It will only be used for the analog stick.

datahead8888 commented 8 years ago

I'm currently thinking the analog jump option should just be removed:

It's not impossible I could be convinced to give this a try and discuss it again, but for now, I am just going to remove it. There are other features I would rather focus on than this.

Quintus commented 8 years ago

Yes, remove it. It is such a niche feature causing so much confusion that it is not worth it.

Vale, Quintus

Quintus commented 8 years ago

Oh, if you close this, please mark it was WONTFIX so the state is clear from the labels.

datahead8888 commented 8 years ago

please mark it was WONTFIX so the state is clear from the labels

Whenever I see "WONTFIX" I assume no action was taken in source control. If we remove the broken feature, that really"is" the fix, so I don't think "WONTFIX" would be appropriate.

datahead8888 commented 8 years ago

If the state is still not clear, I think we should just update the issue's title.