Skip to content

fix: make sidebar/drawer/modal usable on the canvas#65

Merged
avanelsas merged 1 commit into
mainfrom
fix/overlay-components-seed-open
Jun 8, 2026
Merged

fix: make sidebar/drawer/modal usable on the canvas#65
avanelsas merged 1 commit into
mainfrom
fix/overlay-components-seed-open

Conversation

@avanelsas

Copy link
Copy Markdown
Owner

Overlay components were broken on the canvas: they dropped invisible, the selection border traced the host box instead of the painted panel, and modal/drawer couldn't be dropped into at all.

Fixes

  • Seed open on drop so all four overlay components (sidebar/drawer/modal/popover) land visible.
  • Selection measures the painted shadow panel via a per-tag selector (meta/probes), with a host fallback when the panel is collapsed (empty docked sidebar).
  • x-sidebar registered as a container (was missing from meta/slots), seeded width:100%, and placement moved from :top-full-width to :flow.
  • x-drawer / x-modal (:host{display:contents}, fixed panel) flattened for the editor only via the public ::part() API so they have a box and are droppable. Scoped to .canvas-host; never exported.
  • Events for sidebar/drawer/modal registered so they surface in the inspector and can be wired in exports.

Gates

clj-kondo 0/0 · cljfmt clean · 863 tests, 0 failures · release 0 warnings.

🤖 Generated with Claude Code

Seed overlay components open on drop, measure the painted panel for
selection, register x-sidebar as a flow container, flatten modal/drawer
overlays in the editor so they're droppable, and register their events.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@avanelsas avanelsas merged commit 2852e66 into main Jun 8, 2026
1 check passed
@avanelsas avanelsas deleted the fix/overlay-components-seed-open branch June 8, 2026 10:02
@avanelsas avanelsas mentioned this pull request Jun 10, 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.

1 participant