Open
Conversation
…lement logic for randomize map feature
…be randomized again upon replay
Contributor
Author
…loop Because the toggle directly controls the rules, and when the rules are changed it changes the toggle, and there's a delay between those 2 events, spamming the toggle could result in a loop where the toggle changes it's value itself over and over on loop
Contributor
|
I think you should change the base branch, because master is for releases, lol |
Contributor
Author
|
Oh sorry didn't thought about it. I merged Nighlty into my branch and there seem to be no conflicts (there was 3 conflicts on the main menu scene but I was able to fix them all), I tested the game after the merge and everything still works. |
The room would tell the name of the randomly chosen map before this update
The background was ugly and the icon was of a wrong size, so I replaced both with some pattern I made on gimp, not great but better than wwhat was here before
Contributor
|
I like how this was done with a random icon on stage select instead of a more generic toggle in something like Game Settings. The icon can be brushed UP but that is not related to the concept. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I added a toggle to Randomize the stage selection.
When in "Randomize" mode, the player is also able to toggle levels on/off from the random level selection.
Because this affects the UI and I think it's easier to work on UI with prefabs (to avoid future merge conflicts if people are working on different part of the main menu), I also made prefabs for some UI elements.
This PR is in answer to the #367 issue