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

Separate Texture List #188

Merged
merged 11 commits into from
Sep 11, 2023
Merged

Separate Texture List #188

merged 11 commits into from
Sep 11, 2023

Conversation

TokisanGames
Copy link
Owner

@TokisanGames TokisanGames commented Aug 19, 2023

This PR extracts the embedded list of textures (formerly called Surfaces) and moves them to a separate resource called Terrain3DTextureList. This resource is just a text list of the textures and settings, so it can be saved in the scene. But it can be saved to an external .tres so it can be shared among other levels (scenes w/ terrain3d).

It has been tested on the demo with 2 textures. We'll be testing Out of the Ashes next with >20 textures and ~7 levels.

  • Renames surfaces -> textures
  • Creates a separate, savable Terrain3DTextureList resource
  • Converts old surfaces data to new separate texture list
  • Updates old storage file version (0.8) to new version (0.83) on save

Fixes #148

@TokisanGames TokisanGames marked this pull request as draft August 19, 2023 09:59
@TokisanGames TokisanGames force-pushed the seperate-surface-list branch 3 times, most recently from d8449f9 to b35ff6c Compare September 9, 2023 15:52
@TokisanGames TokisanGames changed the title WIP: Separate Surface List Separate Surface List Sep 10, 2023
@TokisanGames TokisanGames marked this pull request as ready for review September 10, 2023 01:02
@TokisanGames TokisanGames changed the title Separate Surface List Separate Texture List Sep 10, 2023
@TokisanGames TokisanGames merged commit dde5a3d into main Sep 11, 2023
@TokisanGames TokisanGames deleted the seperate-surface-list branch September 11, 2023 08:40
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.

Separate Surface List resource
1 participant