Skip to content

Crocomire Escape: fix mapTileMask, item in top-left tile - #2973

Merged
kjbranch merged 1 commit into
vg-json-data:masterfrom
dragonsinth:crocmask
Sep 9, 2026
Merged

Crocomire Escape: fix mapTileMask, item in top-left tile#2973
kjbranch merged 1 commit into
vg-json-data:masterfrom
dragonsinth:crocmask

Conversation

@dragonsinth

@dragonsinth dragonsinth commented Sep 9, 2026

Copy link
Copy Markdown
Contributor
Screenshot 2026-09-09 at 9 44 00 AM

https://maprando.com/seed/YDCh8tKC7/data/visualizer/index.html

Step 4: the upper left tile of Crocomire Escape is marked as not reachable; but that's where the item is.

Copilot AI lite review requested due to automatic review settings September 9, 2026 14:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is a minimal, localized data correction that matches existing mapTileMask conventions in the file and has no apparent downstream risk.

Pull request overview

Adjusts the Crocomire Escape room’s item-node map tile masking so the item displays/anchors correctly in the top-left map tile (as shown in the screenshot), aligning the room’s map representation with the intended in-game placement.

Changes:

  • Update the Item node’s mapTileMask to include the top-left tile in the highlighted mask.
File summaries
File Description
region/norfair/west/Crocomire Escape.json Fixes the Item node’s mapTileMask so the item occupies the correct top-left map tile.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kjbranch
kjbranch merged commit f0a9903 into vg-json-data:master Sep 9, 2026
1 check passed
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.

3 participants