Raise room-envelope height caps from 6m to 20m - #642
Merged
Conversation
Level height, wall custom height, ceiling custom height, and column height were all UI-capped at 6m, which rules out tall single-space builds (climbing gyms, warehouses, halls). 20m covers the tallest realistic single-storey uses — lead-climbing walls top out around 15-18m — while keeping the sliders usable in the common 2-4m band. The caps were never enforced in core schemas; only the controls change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Level height, wall custom height, ceiling custom height, and column height were all UI-capped at 6m, which rules out tall single-space builds (climbing gyms, warehouses, halls).
Changes
20m covers the tallest realistic single-storey uses — lead-climbing walls top out around 15–18m — while keeping the sliders usable in the common 2–4m band. The caps were never enforced in core schemas; only the controls change. Slab elevation, doors, and dormer fields are deliberately untouched.
Walls/ceilings in "Follows level" mode already track the level top, so raising the level height is all a plain tall room needs.
🤖 Generated with Claude Code
Note
Low Risk
UI-only slider/descriptor max changes; schemas and core clamping behavior are unchanged, so risk is limited to allowing taller geometry via existing controls.
Overview
Raises the UI height caps for tall single-space builds (warehouses, halls, climbing gyms) from 6m → 20m.
Updates the level-height slider in the floating level selector, plus custom-height controls and parametric descriptors for walls, ceilings, and columns. Ceiling custom height remains clamped to the level-top bound. Core schemas were never enforcing the old 6m limit—only the controls change.
Reviewed by Cursor Bugbot for commit 6e8b402. Bugbot is set up for automated code reviews on this repo. Configure here.