Open
Conversation
stavrosfa
approved these changes
Feb 19, 2026
Contributor
stavrosfa
left a comment
There was a problem hiding this comment.
Thanks! I tested it with a few maps, looks good to me.
Contributor
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
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.


Fixes #107, missing river delta textures. See also Draft PR #433 about a tile map variant, linked to #407.
There are two river textures in the Civ3 Art assets, a regular set and one for river deltas, to be drawn when the river meets the sea. The delta textures are not currently being loaded in.
This patch adds the textures, and applies according to a simple heuristic. Given the close relationship between the two tile sets and how they are deployed, the implementation reduces to a simple check to see whether the tile drawing is happening next to water tiles or not.
Sample save file
Given how closely related to the regular river tiles the delta tiles are, regardless of how the tile map effort proceeds, #433 can probably be closed off.
--
Note the non-delta river segments in the delta texture:
