Skip to content

Conversation

@ZenithC7
Copy link
Contributor

Also included 5Hex tile resizing to the standard 120x105 tile sizing, to avoid edge bleeds when displaying the tiles.

+ Added Plantation, Coconut, Dry Dock, Hardwood, Rubber
@SeventhM
Copy link
Collaborator

SeventhM commented Aug 13, 2025

Is it intended that 5Hex Perfume+Plantation uses the HR sprite or that a number of the 5Hex sprites got slightly updated looks? I assume the former is accidental, the latter intended?

@SeventhM
Copy link
Collaborator

Actually, ignore the second question, missed the commit history. First one is valid, though

@ZenithC7
Copy link
Contributor Author

I'm was experimenting on whether to also update the 5Hex plantation design on both the basegame (on 5H repo) because the current one was ugly, and left Perfume+Plantation there bc it looks much better than leaving it blank (and get replaced with an even uglier FantasyHex plantation)

@SeventhM
Copy link
Collaborator

I mean sure I guess, fair enough, maybe, but that's not my point

My point is that according to the diff, there was already a 5Hex styled Perfume+Plantation. This PR replaces it with the HR Perfume+Plantation, which looks incorrect. I would assume that if you intended to replace it later, it would still at least use the current 5Hex image under the current 5Hex style, not the HR image that's obviously not matching

@SeventhM
Copy link
Collaborator

Like, you can still replace the plantation design later, that's fine, but updating only 1 of the plantation images when you're already editing them all to meet with 5Hex's image sizes seems weird. I would imagine you either switch all of them to HR or change none of them. Not randomly only change 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants