Skip to content

Feat/stairs fence update#226

Merged
Aymericr merged 9 commits intopascalorg:mainfrom
sudhir9297:feat/stairs-fence-update
Apr 14, 2026
Merged

Feat/stairs fence update#226
Aymericr merged 9 commits intopascalorg:mainfrom
sudhir9297:feat/stairs-fence-update

Conversation

@sudhir9297
Copy link
Copy Markdown
Contributor

What does this PR do?

Adds a new Fence build element and introduces major stair system improvements, aligning both with plan3D behavior and the project’s 3D editor workflows.

This PR includes

Fence system

  • New fence scene node/schema with parametric properties
  • 3D fence drawing in build mode with wall-like snapping
  • Fence rendering in the viewer/editor
  • Selection, move, delete, duplicate, and scene-tree support
  • Sidebar panel for editing fence parameters
  • Linked endpoint behavior for connected fences (live updates on move)
  • Toolbar/sidebar integration with fence icon

Stair system

  • Stair-level railing controls with rendering for straight stairs
  • Fixes for railing mapping and landing behavior
  • Railings update live while editing
  • New curved and spiral stair types with full rendering + controls
  • Fit To Floor support for curved stairs and improved panel logic
  • Spiral stair features (center column, supports, optional top landing)
  • Fixed stair duplication and improved stability
  • Runtime crash fixes for invalid wall children data

How to test (summary)

  • Draw and edit fences; verify snapping, parameters, and linked behavior
  • Test fence actions (move, duplicate, delete) and scene-tree presence
  • Create and modify stairs (straight, curved, spiral); verify railings and controls
  • Switch stair types live and test duplication/movement
  • Validate no regressions in wall/door/window interactions

sudhir9297 and others added 9 commits April 13, 2026 11:47
…reeNode

TreeNode was passing `node` (undefined variable) instead of `nodeId` to
WallTreeNode and FenceTreeNode, causing a runtime ReferenceError.
Updated FenceTreeNode to accept nodeId and look up the node from the
scene store internally, consistent with all other tree node components.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Aymericr Aymericr merged commit a205e4f into pascalorg:main Apr 14, 2026
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.

2 participants