Skip to content

fix: resolve several bugs in the tkinter ui - #124

Merged
Joltras merged 5 commits into
devfrom
fix/ui-bugs
Nov 2, 2026
Merged

Joltras merged 5 commits into
devfrom
fix/ui-bugs

Conversation

@Joltras

@Joltras Joltras commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

Proposed changes

The canvas was sized with the room width squared, making it over 14000 pixels wide, and now matches the floor size. The right arrow key bypassed the generator's floor tracking, so saving afterwards stored the previous floor, and the menu shortcuts were only displayed but never bound. Pressing enter in the next seed field only changed the displayed seed without generating anything, and now generates a floor with the entered seed. Step by step drawing now uses the tkinter event loop instead of a background thread, which avoids calling tkinter from another thread and stops the previous drawing when a new floor is generated.

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 3dc9309 into dev Nov 2, 2026
4 checks passed
@Joltras
Joltras deleted the fix/ui-bugs branch November 2, 2026 11:59
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