[Ultica] New hoisted versions of default pride & state flags #2473
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.
Summary
The hoisted American flag looks great, but I'd love to see even more! Unfortunately, it seems that currently only the "base" versions of items with variants are supported (e.g. "pride_flag" in layering.json triggers for all variants such as the trans flag, binary flag, etc.). This PR would at least support the defaults for the "pride flag" and "state flag" items, and someday I'd hope to include the rest of the variants if they're ever supported.
Content of the change
Two new sprites created using the existing hoisted flag .xcf template, plus an edit to Ultica's layering.json to enable them to "fly" on metal & wooden flagpoles.
Testing
Built from source (experimental-2024-08-06-0149 & last CDDA-Tilesets) and confirmed that the American, rainbow pride, and Massachusetts flags all "fly" correctly on both types of flagpoles.
Additional information