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

Update worldmap tiles to fit art style + deprecation #2735

Merged
merged 20 commits into from
Sep 10, 2024

Conversation

Rusty-Box
Copy link
Member

@Rusty-Box Rusty-Box commented Jan 14, 2024

This PR aims to updates all worldmap tiles to make sure tha even they now are consitent with all the other assets, along side a couple missing additions, like the Corrupted Underground. It also deprecates a bunch of old assets!

TO-DO:

  • Snow
  • Ice Cave
  • Forest + Corrupted Forest
  • Underground Forest + Corrupted Underground
  • Granito Houses + Ruin Structures
  • Corrupted Granito Houses + Corrupted Ruin
  • Corrupted Roots
  • Paths
  • Water

  • Proper deprication of worldmap tiles and assets
  • Autotiling for all compatible worldmap tilesets (big thanks to @tylerandari13)

@Rusty-Box Rusty-Box added type:patch status:in-progress Progress has been done, but more is intended be done category:design labels Jan 14, 2024
@Rusty-Box Rusty-Box added this to the 0.7.0 milestone Jan 14, 2024
@VincentAraneo
Copy link

What a great art you make Rusty!😁

@Rusty-Box Rusty-Box added status:needs-review Work needs to be reviewed by other people and removed status:in-progress Progress has been done, but more is intended be done labels Apr 3, 2024
@Rusty-Box Rusty-Box marked this pull request as ready for review April 3, 2024 19:18
@Brockengespenst
Copy link
Contributor

Looks pretty fine so far. I just saw three small glitches:
iceworld
rooted-forest

@Rusty-Box
Copy link
Member Author

That is intended. I removed the paths from the igloo and latter tiles as it is more handy to not have path variants for each. This has been addressed for the new world maps that are currently in the works but I can fix that for the old ones as well for the time being.

@Rusty-Box
Copy link
Member Author

Those should be fixed now.

@Brockengespenst
Copy link
Contributor

Those should be fixed now.

Alright, looks good now. :-)

@Rusty-Box Rusty-Box requested a review from Vankata453 May 5, 2024 18:42
@Rusty-Box
Copy link
Member Author

@Vankata453 You know better how this whole depreciation thing works? Can you tell me if I did it right, cuz I personally think I probably have.

@Vankata453
Copy link
Member

Vankata453 commented May 5, 2024

Cannot fully check this now, but one thing you should do is move all the .deprecated tile image files into a deprecated/ sub-directory, under the original folder of the image.

@Rusty-Box
Copy link
Member Author

Cannot fully check this now, but one thing you should do is move all the .deprecated tile image files into a deprecated/ sub-directory, under the original folder of the image.

@Vankata453 Do I still give those files the ".deprecated"-file extension then, when I put them in a deprecated/ sub-directory?

@Vankata453
Copy link
Member

@Vankata453 Do I still give those files the ".deprecated"-file extension then, when I put them in a deprecated/ sub-directory?

Actually, sorry, no. For tiles, the "deprecated" directory is enough.

@Rusty-Box
Copy link
Member Author

Gonna add corrupted tower, ruin and Granito houses as well cuz I forgot that I also need that for story mode

@tobbi tobbi merged commit bd250ef into SuperTux:master Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:design status:needs-review Work needs to be reviewed by other people type:patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants