Skip to content

feat(mapgen-studio): disable canvas controller and grab cursor pre-run#1639

Merged
mateicanavra merged 1 commit into
mainfrom
design/prerun-cursor
Jun 12, 2026
Merged

feat(mapgen-studio): disable canvas controller and grab cursor pre-run#1639
mateicanavra merged 1 commit into
mainfrom
design/prerun-cursor

Conversation

@mateicanavra

Copy link
Copy Markdown
Owner

Before a run completes, the map canvas displayed a grab cursor even though dragging moved nothing visible — the deck controller was active but the only pre-run texture is a CSS graticule behind the canvas, not a deck layer. This gave users a misleading affordance.

DeckCanvas now accepts an interactive prop (defaults to true) that gates both the deck controller and the cursor. When false, the controller is disabled and the cursor is default; when true, the controller (including the moveSpeed: 150 keyboard tuning) and the grab/grabbing cursor are active. Transitions between states apply via setProps so the Deck instance is never remounted.

CanvasStage passes interactive={hasManifest}, using the same boolean gate that controls the "Awaiting matter" overlay, so the cursor and camera interactivity become live exactly when there is rendered matter to interact with.

This was referenced Jun 12, 2026
@github-actions

Copy link
Copy Markdown

Railway preview (MapGen Studio): not provisioned for this PR.

Policy (Graphite stacks): previews are created only for the top-of-stack PR by default.

  • To force a preview for this PR: add label railway-preview
  • To suppress a preview: add label no-railway-preview

Debug: {"isClosed":false,"isFork":false,"suppress":false,"force":false,"isTopOfStack":false,"hasToken":true,"draft":false}

mateicanavra commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

  • Jun 12, 8:50 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 12, 10:01 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 12, 10:01 PM UTC: @mateicanavra merged this pull request with Graphite.

@mateicanavra
mateicanavra changed the base branch from design/tile-orientation to graphite-base/1639 June 12, 2026 21:58
@mateicanavra
mateicanavra changed the base branch from graphite-base/1639 to main June 12, 2026 21:59
@mateicanavra
mateicanavra force-pushed the design/prerun-cursor branch from c42a9f1 to 507cd4b Compare June 12, 2026 22:00
@mateicanavra
mateicanavra merged commit b2264bc into main Jun 12, 2026
This was referenced Jun 12, 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