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] Disable input in CharSelect until stayFunky plays #3355

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

ActualMandM
Copy link
Contributor

@ActualMandM ActualMandM commented Sep 17, 2024

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

#3156

Briefly describe the issue(s) fixed.

This PR introduces a new allowInput boolean to CharSelectSubState, false by default and is only set to true when we start playing stayFunky. This prevents the game from being able to run/cancel tweens on a currently null object, being FlxG.sound.music.

Include any relevant screenshots or videos.

Before

2024-09-17_15-49-58_Funkin.mp4

After

2024-09-17_15-52-51_Funkin.mp4

@EliteMasterEric EliteMasterEric added the status: pending triage The bug or PR has not been reviewed yet. label Sep 18, 2024
@EliteMasterEric EliteMasterEric added type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression 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 Sep 18, 2024
@EliteMasterEric EliteMasterEric self-assigned this Sep 18, 2024
@EliteMasterEric
Copy link
Member

approved

@EliteMasterEric EliteMasterEric added this to the 0.5.1 milestone Sep 18, 2024
@AbnormalPoof AbnormalPoof mentioned this pull request Sep 23, 2024
@ninjamuffin99 ninjamuffin99 merged commit 9708165 into FunkinCrew:develop Sep 30, 2024
@ActualMandM ActualMandM deleted the bugfix/charselect-crash branch October 1, 2024 04:11
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: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants