Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add swimming #1407

Merged
merged 23 commits into from
May 31, 2020
Merged

Add swimming #1407

merged 23 commits into from
May 31, 2020

Conversation

weluvgoatz
Copy link
Member

Here is the revived branch of swimming, which is a planned feature. The code and sprites were created mainly by @Zwatotem and me. Although I have tested and fixed numerous bugs with this feature, it would be nice if anyone can find any that I missed and maybe then this could be merged? Swimming would fit right in the game anyway.

@Ordoviz
Copy link
Contributor

Ordoviz commented May 24, 2020

Is it intentional that you can spam the jump button to gain hyper speed? I suggest limiting Tux’ speed.

Issues:

General:

  • Hold up while you are in water. Eventually, Tux will rotate on the water surface.
  • When Tux jumps into water from above, the sprite flips out for a frame. This makes the transition not very smooth.

Big Tux:

  • Big Tux teleports a little when his hitbox changes from horizontally to vertically and vice versa.
  • Big Tux can buttjump when he makes a high jump out of the water. This looks really odd.

Fire/ice flower:

  • The bullets get shot to the right or left not the direction Tux is facing. We also need to discuss how Tux creates these bullets and how he throws them (with the right movement it looks like Tux poops them out).

Small Tux:

  • When small Tux jumps out of the water to the top left, he flips after he reaches his max height and falls down on his back. This is nice for falling straight down but not for jumping in a parabola (it does not happen with big Tux or when jumping to the top right)

Missing/bad graphics:

  • Big Tux looks very janky. I cannot take him seriously.
  • Create a sound effect that plays when pressing JUMP while in water and make Tux swirl his arms. Some water bubbles would also be nice.

Edit: I almost forget: Thank you for the great work! Swimming and sliding is exactly what SuperTux needs and what makes the game fun.

@Narre
Copy link
Contributor

Narre commented May 24, 2020

I've been working on a new add-on for SuperTux for a while now (a.k.a since 2015), and I have a good few levels centred around swimming, so I will test this feature in actual levels (rather than test levels) as soon as I can, and I will make sure to leave as much feedback as I can.

@tobbi tobbi marked this pull request as draft May 27, 2020 10:07
Copy link
Contributor

@christ2go christ2go left a comment

Choose a reason for hiding this comment

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

I hope, these comments might help making the pr even better. I didn't test it yet, so this is mostly code-style/code-organization related.

@christ2go christ2go marked this pull request as ready for review May 28, 2020 08:06
@weluvgoatz
Copy link
Member Author

I've fixed some bugs with this PR. I marked them off that checklist, as well as some suggestions that aren't necessary for this PR to be merged/things I can't replicate or reproduce or see as a problem.

@Ordoviz
Copy link
Contributor

Ordoviz commented May 30, 2020

Your new changes removed Tux’ backflip animation.

@weluvgoatz
Copy link
Member Author

weluvgoatz commented May 30, 2020 via email

@Grumbel
Copy link
Member

Grumbel commented Apr 10, 2021

What's the plan with the backflip? I kind of liked the old behavior and haven't really seen much reason to change it.

@weluvgoatz
Copy link
Member Author

@Grumbel: At some point, the backflip will be redrawn to look better, as the rotating behavior definitely has room for improvement (a hand drawn flipping animation is ideal) (would be part of the Tux graphics revamp, see #1110 for more). However, if a stable release happens before the graphics are updated, then the old behavior will probably be temporarily re-added.

@mrkubax10 mrkubax10 deleted the feature/swimming2 branch August 18, 2023 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants