Skip to content

docs(mapgen-studio): record Pass-5 design fixes (toolbar v2, tile orientation, grid icon, flash guard, pre-run cursor)#1641

Merged
mateicanavra merged 1 commit into
mainfrom
design/pass5-ledger
Jun 12, 2026
Merged

docs(mapgen-studio): record Pass-5 design fixes (toolbar v2, tile orientation, grid icon, flash guard, pre-run cursor)#1641
mateicanavra merged 1 commit into
mainfrom
design/pass5-ledger

Conversation

@mateicanavra

Copy link
Copy Markdown
Owner

Records the completion of Pass-5 design fixes for the mapgen studio redesign, covering five implemented slices:

  • Toolbar architecture v2: Top bar restructured as the Game toolbar with a Gamepad2 identity, saved-config selector, inlined GameConsole cluster, and a game-setup disclosure for Leader/Civ/Difficulty/Speed; bottom bar established as the World/Map console with Globe identity, Size/Players/Resources alongside Seed, and a History icon compressing last-run stats
  • Grid icon restoration: Fixed a ViewControls grid toggle that was rendering an empty div instead of the Grid3x3 icon, with a categorical sweep confirming no other empty placeholders exist
  • Tile orientation correction: Odd-Q tiles were drawing pointy-top on a sheared lattice and now render flat-top on the canonical hex space used by the Delaunay/world.xy layers, with correct transparency and border behavior for noData/non-finite tiles
  • Flash fix: Eliminated an unstyled root element during reload (~138ms window) by placing background and color-scheme on :root for both theme branches inline, bringing dark root rendering down to ~8ms
  • Pre-run cursor honesty: Canvas is now inert with a default cursor before a manifest exists, replacing a grab cursor that promised an invisible drag; grab/grabbing and the deck controller are restored post-run via setProps without remounting

Also records a new standing rule: search existing component libraries (shadcn/ui first) before building custom UI patterns from scratch. All five slices verified on :5173 with 167 passing tests and five strict OpenSpec validations.

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:06 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 12, 10:06 PM UTC: @mateicanavra merged this pull request with Graphite.

@mateicanavra
mateicanavra changed the base branch from design/flash-fix to graphite-base/1641 June 12, 2026 22:02
@mateicanavra
mateicanavra changed the base branch from graphite-base/1641 to main June 12, 2026 22:04
@mateicanavra
mateicanavra force-pushed the design/pass5-ledger branch from a61ae24 to c37accd Compare June 12, 2026 22:05
@mateicanavra
mateicanavra merged commit 42c4f45 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