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

Rewrite map fixes #3324

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

Conversation

jetrotal
Copy link
Contributor

@jetrotal jetrotal commented Jan 6, 2025

Fixed the issues described at #3323


Also I Discovered one last quirk, from toggling autotiles ON/OFF:

  • On EasyRPG the autotile is changing a buch of its own neighbour tiles.
  • On maniacs it changes only the closest to itsef.

This Quirk is not a dealbreakers, so didn't look at the code of it yet.


Things are now very closer to the Maniacs version now, only missing that fix and the AB lower tiles support.

- Added a more robust way of identifying TILE_BLOCKS. This fixes the issue where most of lower tiles were not being aplied.
- Reduced an output warning that repeated itself twice.
- Fixed wrong IDs
- Fixed wrong Layer assertion
- Added a conditional branch to protect IDs from overflowing.
@Ghabry Ghabry added this to the 0.8.1 milestone Jan 6, 2025
@Ghabry
Copy link
Member

Ghabry commented Jan 7, 2025

Web version for easier testing:

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

Successfully merging this pull request may close these issues.

2 participants