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

[FIX] FreeplayState no longer loads all songs when exiting Erect/Nightmare songs #3711

Closed

Conversation

KoloInDaCrib
Copy link
Contributor

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

3664 3706

Briefly describe the issue(s) fixed.

Changes the filter part of generateSongList function to use rememberedDifficulty instead of currentDifficulty.
Also makes the rememberedDifficulty reset when entering CharSelectSubState because of the case of user having selected Erect/Nightmare and then switching to Pico, who does not have Erect/Nightmare difficulties (as far as we know)

Include any relevant screenshots or videos.

2024-10-15.22-25-16.mp4

EliteMasterEric and others added 2 commits October 11, 2024 23:57
remind me to delete some of my fork branches its getting too crowded
@github-actions github-actions bot added medium A medium pull request with 100 or fewer changes haxe Issue/PR modifies game code documentation Issue/PR modifies documentation or README files labels Oct 15, 2024
@KoloInDaCrib
Copy link
Contributor Author

eric i beg of you please give the docs of the develop branch a check i haven't touched them for this pr !!! >:[

@AbnormalPoof
Copy link
Contributor

You can always rebase and remove the commit

@KoloInDaCrib KoloInDaCrib deleted the freeplay-song-loading-fix branch November 6, 2024 20:23
@KoloInDaCrib KoloInDaCrib restored the freeplay-song-loading-fix branch November 6, 2024 20:23
@KoloInDaCrib KoloInDaCrib deleted the freeplay-song-loading-fix branch November 30, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issue/PR modifies documentation or README files haxe Issue/PR modifies game code medium A medium pull request with 100 or fewer changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants