Skip to content

fix: correct teleport boss room placement - #120

Merged
Joltras merged 2 commits into
devfrom
fix/teleport-boss-room
Nov 2, 2026
Merged

Joltras merged 2 commits into
devfrom
fix/teleport-boss-room

Conversation

@Joltras

@Joltras Joltras commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Proposed changes

When the boss room had to be moved behind a teleport room, the generator checked one corner but moved the room into a different one, so the boss room could end up next to other rooms or on top of an existing one. The moved boss room is now placed in the corner that was actually checked, or in any other free cell without neighbours, and is registered in the floor grid. If no such cell exists, the boss room stays in place and no teleport room is added. Additionally, TkinterTeleportRoom now stores the connected room id correctly instead of the room type, which was caused by its base class order.

Types of changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

@Joltras
Joltras merged commit 30d099c into dev Nov 2, 2026
4 checks passed
@Joltras
Joltras deleted the fix/teleport-boss-room branch November 2, 2026 11:11
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.

1 participant