Skip to content

Fixed Up Arrow Jump Bug#178

Open
videozane wants to merge 1 commit into
mainfrom
Up-arrow-jump-bug
Open

Fixed Up Arrow Jump Bug#178
videozane wants to merge 1 commit into
mainfrom
Up-arrow-jump-bug

Conversation

@videozane
Copy link
Copy Markdown
Collaborator

@videozane videozane commented May 27, 2022

Fixed bug where you would just jump forever, I believe caused by Micah (sorry playa)

  • Go into player.js and fix jump() method (likely caused by looping code)

To recreate in main just press up arrow once it'll keep jumping

Fixed bug where you would just jump forever.
@videozane videozane changed the title Fixed Code Fixed Up Arrow Jump Bug May 27, 2022
@videozane videozane self-assigned this May 28, 2022
@ethanbrose ethanbrose self-requested a review May 29, 2022 01:12
Copy link
Copy Markdown
Contributor

@ethanbrose ethanbrose left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown

@DonovanTyler DonovanTyler left a comment

Choose a reason for hiding this comment

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

While holding the jump arrow still causes the player to jump repeatedly (unlike holding Z), the bug of jumping forever once the up arrow is pressed seems to have been fixed. All this being said, I tried recreating the bug in main and I couldn't do so (though in this version, the up arrow appears to have the functionality of the Z key in main rather than the up arrow in the previous version). As such, though I am approving this pull request since the code is functional, I suggest more reviews be done before actually going through with it.

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.

3 participants