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

[BUGFIX] Stop input spam on character select #3398

Merged

Conversation

ACrazyTown
Copy link
Contributor

@ACrazyTown ACrazyTown commented Sep 20, 2024

Does this PR close any issues? If so, link them below.

None as far as I know?

Briefly describe the issue(s) fixed.

Currently if you hold a key down and press enter on a character in the character select screen, the hold won't stop and the cursor will keep moving.

In some cases this can lead to the current character to be set to an invalid value and the game will crash.

Include any relevant screenshots or videos.

8mb.video-jkb-Aa2vrZFM.mp4

@ACrazyTown ACrazyTown changed the base branch from main to develop September 20, 2024 16:22
@EliteMasterEric
Copy link
Member

Is this a duplicate of #3355?

@ACrazyTown
Copy link
Contributor Author

Is this a duplicate of #3355?

Afaik no, that PR seems to block any inputs until the music starts playing. My PR sets the spam variables to false when a character is selected

@EliteMasterEric EliteMasterEric added the status: pending triage The bug or PR has not been reviewed yet. label Sep 22, 2024
@ACrazyTown ACrazyTown changed the title Stop input spam on character select [BUGFIX] Stop input spam on character select Sep 22, 2024
@ninjamuffin99 ninjamuffin99 force-pushed the fix/charselect-spam-hold branch from ef6a171 to 2586073 Compare September 30, 2024 22:13
@ninjamuffin99 ninjamuffin99 force-pushed the fix/charselect-spam-hold branch from 2586073 to 435939a Compare September 30, 2024 22:17
@ninjamuffin99 ninjamuffin99 merged commit c255f35 into FunkinCrew:develop Sep 30, 2024
@ninjamuffin99
Copy link
Member

thank you very much :)

@EliteMasterEric EliteMasterEric added type: minor bug Involves a minor bug or issue. status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: pending triage The bug or PR has not been reviewed yet. labels Oct 1, 2024
@ACrazyTown ACrazyTown deleted the fix/charselect-spam-hold branch October 5, 2024 19:34
@EliteMasterEric EliteMasterEric added this to the 0.5.2 milestone Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants