Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding more sand borders in floortiles #855

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Adding more sand borders in floortiles #855

wants to merge 2 commits into from

Conversation

Seyjin
Copy link
Contributor

@Seyjin Seyjin commented Mar 8, 2012

Tiles were missing for several sandborders in floortiles. I have added.
I did not see any problem in game due to the change in image size.

@Maescool
Copy link
Owner

Maescool commented Mar 8, 2012

this needs to be implemented in the code, so the tiles are used by the game :)

@Ampa
Copy link

Ampa commented Mar 9, 2012

I would have thought that this should actually be implemented in the same way as shadows... the sand being a transparent overlay that can be put on top of neighbouring tiles.

This would allow transitions between sand and any other floor type that is introduced in the future, and requires fewer component edges and corners as the various combinations can be constructed on the fly.

@master-lincoln
Copy link
Collaborator

agree to @Ampa

@Seyjin
Copy link
Contributor Author

Seyjin commented Mar 10, 2012


or by using rotations

@Delvar
Copy link
Contributor

Delvar commented Mar 15, 2012

+1 @Ampa
on an expansion to this idea could we separate each floor tile to its own texture file which would include generic transitions.
so at a later date we could add QuickSand, CutStone, Dirt, Grass or RoughStone etc and have them transition from one to another without having an exponential increase in required textures.
all we would need to do is give each Tile a priority value so we always know which way to do the transition.

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

Successfully merging this pull request may close these issues.

5 participants