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] Fix the instrumental selector being spammable #3866

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

AbnormalPoof
Copy link
Contributor

@AbnormalPoof AbnormalPoof commented Nov 15, 2024

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

Fixes #3562

Briefly describe the issue(s) fixed.

CapsuleOptionsMenu has its own method of handling inputs, independent of FreeplayState. This, in turn, meant that inputs were still enabled even AFTER selecting a song.

This PR adds a Boolean called busy. This pretty much acts the same as the one in FreeplayState.

This is sort of a janky solution, but you can't really do anything much on the Freeplay side since busy is enabled to prevent FreeplayState from accepting inputs while the Instrumental Selector is open (unless you like... made a new boolean to check if the instrumental selector is open or something lol).

Something to note

The arrows in the Instrumental Selector (InstrumentalSelector) can still be controlled, however, this is purely a visual thing.
I tried tackling the issue before opening this PR, but I couldn't really figure out a solution without it being a mess.

@github-actions github-actions bot added the medium A medium pull request with 100 or fewer changes label Nov 15, 2024
@github-actions github-actions bot added haxe Issue/PR modifies game code and removed medium A medium pull request with 100 or fewer changes labels Nov 15, 2024
@Honton129
Copy link

si please i need this to be merged so bad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
haxe Issue/PR modifies game code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants