Skip to content

Always clear the companion box layer once it has nothing to show - #1981

Merged
BryonLewis merged 1 commit into
mainfrom
dev/companion-box-ghost
Sep 25, 2026
Merged

BryonLewis merged 1 commit into
mainfrom
dev/companion-box-ghost

Conversation

@mattdawkins

Copy link
Copy Markdown
Member
  • The layer manager skipped boxEditLayer.disable() whenever the companion was already in disabled mode, to avoid a redundant mode(null). GeoJS puts the layer in that mode itself when editing ends (right-click, completion) while keeping the completed box, so the box survived as a ghost on every frame, with GeoJS's hover highlight on it even when nothing was selected
  • Clear finalized edit overlays to prevent ghost boxes and hover highlights #1961 made disable() clear overlays without the redundant mode(null), so calling it unconditionally is now safe; this removes the remaining bypass
  • Seen on web stereo data after a line's transfer to the other camera failed; same change is on the Add SAM2/SAM3 browser segmentation and stereo auto-population #1672 branch

🤖 Generated with Claude Code

GeoJS ends editing on its own (right-click, completion) and leaves the
box behind in disabled mode; skipping disable() there kept it as a
ghost on every frame with a hover highlight.
@BryonLewis
BryonLewis merged commit 9cceb53 into main Sep 25, 2026
3 checks passed
@BryonLewis
BryonLewis deleted the dev/companion-box-ghost branch September 25, 2026 18:10
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