Create centralized space to limit dye colors to only vanilla ones whe… #6335
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.
This PR fixes two issues that occur when playing with mods that add more dye colors:
Utils.javaallow these new dyes to color conveyors and balloonsEspecially the second change allows it to work with mods like dye depot without simply crashing on startup in many different ways.
What this PR does not fix and is in no way the responsibility of IE is missing data & assets for chutes and sheet metals.
I have still allowed them to be automatically created by looping through
DyeColors.values()instead of only using vanilla colors here. This is in order for other mods, specifically Dye The World to add these assets and data and allow this compatibility