Skip to content

fix(mapgen-studio): tile grid reads at every zoom — fill-derived border ink + grid-preferring layer default#1661

Merged
mateicanavra merged 1 commit into
mainfrom
design/y3-tile-grid-ink
Jun 12, 2026
Merged

fix(mapgen-studio): tile grid reads at every zoom — fill-derived border ink + grid-preferring layer default#1661
mateicanavra merged 1 commit into
mainfrom
design/y3-tile-grid-ink

Conversation

@mateicanavra

Copy link
Copy Markdown
Owner

The grid disappeared for every stage besides the first: X6's constant
graphite border ink (#0d0d11 — the page substrate) is invisible between
dark fills at fit zoom, where Huge-map tiles are a few pixels wide; the
tessellation dissolved into disconnected dots. Only the Delaunay mesh
(its own slate ink) still read as a grid, hence 'first stage only'.

  • The one tile-border RULE replaces the constant ink: border = the tile's
    OWN fill x 0.55, fully opaque (tileBorderColorForFill) — darker than its
    fill BY CONSTRUCTION, legible at every zoom, both themes, every palette,
    and still etched-grout up close. Mesh contract unchanged (unfilled tiles
    draw nothing). Applied to both grid and gridFields strokes.
  • Layer-selection default prefers the step's tile-space GRID layer over
    whichever layer the worker emitted first (points/mesh): the map studio
    defaults to the map when selection resets across stage/step switches.

New border-rule pin in tileOrientation.test.ts (per-tile derived stroke,
opaque, distinct between tiles). Verified live across stages 2/10/13:
contiguous hex lattice at fit zoom; stage 10 now defaults to the Biome
Index grid instead of centroid points.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

mateicanavra commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Jun 12, 2026

mateicanavra commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

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

@mateicanavra
mateicanavra changed the base branch from design/y2-saved-config-precedence to graphite-base/1661 June 12, 2026 22:54
@mateicanavra
mateicanavra changed the base branch from graphite-base/1661 to main June 12, 2026 22:55
…er ink + grid-preferring layer default

The grid disappeared for every stage besides the first: X6's constant
graphite border ink (#0d0d11 — the page substrate) is invisible between
dark fills at fit zoom, where Huge-map tiles are a few pixels wide; the
tessellation dissolved into disconnected dots. Only the Delaunay mesh
(its own slate ink) still read as a grid, hence 'first stage only'.

- The one tile-border RULE replaces the constant ink: border = the tile's
  OWN fill x 0.55, fully opaque (tileBorderColorForFill) — darker than its
  fill BY CONSTRUCTION, legible at every zoom, both themes, every palette,
  and still etched-grout up close. Mesh contract unchanged (unfilled tiles
  draw nothing). Applied to both grid and gridFields strokes.
- Layer-selection default prefers the step's tile-space GRID layer over
  whichever layer the worker emitted first (points/mesh): the map studio
  defaults to the map when selection resets across stage/step switches.

New border-rule pin in tileOrientation.test.ts (per-tile derived stroke,
opaque, distinct between tiles). Verified live across stages 2/10/13:
contiguous hex lattice at fit zoom; stage 10 now defaults to the Biome
Index grid instead of centroid points.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mateicanavra
mateicanavra force-pushed the design/y3-tile-grid-ink branch from 947e6f7 to cac39c6 Compare June 12, 2026 22:56
@mateicanavra
mateicanavra merged commit e6eb282 into main 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