Level pill: no ground-floor delete; hide compass in studio - #723
Merged
Conversation
The floating level selector's kebab offered Delete for the ordinal-0 ground floor — the vertical model's zero anchor — while the site panel and command palette already guard it; disable it with the same rule and explanatory title. The floorplan compass portaled into the viewer area outlived the studio stage overlays (gallery, materials, item builder); gate it on the edit workspace like the rest of the editor chrome. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SrRkPvd6AogVAbt3PNeLAw
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
1 participant
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.
Replaces #722 (auto-closed when its stacked base branch was deleted by the #721 merge; now rebased onto main).
What
level.level !== 0) and the same explanatory title as the site panel.display:none), which also let it paint above the studio stage overlays — it showed in the gallery/materials/item-builder tabs. Now gated on the edit workspace, same pattern asHelperManager.Verification
Playwright probe against the running community app: Ground Floor's Delete item renders disabled with the tooltip; compass hidden after switching to Studio; compass returns in Edit.
Note for reviewers
There is no store-layer guard for ground-floor deletion —
deleteNode/deleteLevelWithFallbackSelection/ MCPdelete-nodeall delete it unconditionally; every guard is UI-level (site panel, command palette, now the pill). A core-level guard may be worth a follow-up.🤖 Generated with Claude Code
https://claude.ai/code/session_01SrRkPvd6AogVAbt3PNeLAw
Note
Low Risk
UI-only guards and visibility gating with no changes to delete APIs or scene store behavior.
Overview
Studio workspace no longer shows the floorplan compass overlay. The compass render is gated on not being in
studioworkspace mode, matching how other editor chrome (e.g. helpers) stays off the renders/materials/item-builder stage.Floating level selector now applies the same ground-floor rule as the site panel and command palette: Delete level is disabled when
level.level === 0, with a tooltip explaining the ground level cannot be deleted. Styling reflects the disabled state.Reviewed by Cursor Bugbot for commit 2ecae46. Bugbot is set up for automated code reviews on this repo. Configure here.