Skip to content

Fix swapped player map / heightmap assignments; add Frostfang and Sylvanreach maps#3

Merged
iaretechnician merged 1 commit into
masterfrom
copilot/fix-island-map-assignments-add-new-maps
May 25, 2026
Merged

Fix swapped player map / heightmap assignments; add Frostfang and Sylvanreach maps#3
iaretechnician merged 1 commit into
masterfrom
copilot/fix-island-map-assignments-add-new-maps

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 25, 2026

Player maps and heightmaps for Aetherfall (mainland) and Crownspire were stored with inverted roles — the heightmap file was being displayed as the player map and vice versa. Also adds map assets for islands #3 (Frostfang) and #4 (Sylvanreach).

Fixes

  • Mainland & Crownspire — swapped file contents between _git/artwork/maps/ and _git/artwork/heightmaps/ so *-player-map.png holds the visible player map and *-heightmap.png holds the terrain generation asset; doc references unchanged

New assets

  • Uploaded files renamed and placed per project convention:
    • A9993BBB-...maps/frostfang-player-map.png
    • IMG_3130.pngheightmaps/frostfang-heightmap.png
    • 152EBE82-...maps/sylvanreach-player-map.png
    • IMG_3132.pngheightmaps/sylvanreach-heightmap.png

Docs

  • Added #### Island Map sections (matching existing format) to Frostfang Isle and Sylvanreach Isle in _git/docs/islands.md

…reach maps

- Swap mainland and crownspire player map / heightmap file assignments
- Add renamed Frostfang player map and heightmap (from uploaded files)
- Add renamed Sylvanreach player map and heightmap (from uploaded files)
- Add Island Map sections for Frostfang Isle and Sylvanreach Isle in islands.md
@iaretechnician iaretechnician marked this pull request as ready for review May 25, 2026 15:02
Copilot AI review requested due to automatic review settings May 25, 2026 15:02
@iaretechnician iaretechnician merged commit 7169d2d into master May 25, 2026
@iaretechnician iaretechnician deleted the copilot/fix-island-map-assignments-add-new-maps branch May 25, 2026 15:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Corrects map asset wiring by ensuring each island’s *-player-map.png is the visible player-facing map and each *-heightmap.png is the terrain-generation heightmap, and extends island documentation/assets for Frostfang and Sylvanreach.

Changes:

  • Swapped the stored contents/roles of Aetherfall (mainland) and Crownspire player maps vs. heightmaps so filenames match their intended usage.
  • Added new map + heightmap assets for Frostfang and Sylvanreach following the existing naming/location convention.
  • Updated _git/docs/islands.md to include “#### Island Map” sections (image + download links) for Frostfang and Sylvanreach.

Reviewed changes

Copilot reviewed 1 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
_git/docs/islands.md Adds Island Map sections (image + download links) for Frostfang and Sylvanreach.
_git/artwork/maps/mainland-player-map.png Ensures the mainland player map file contains the actual player map (not the heightmap).
_git/artwork/heightmaps/mainland-heightmap.png Ensures the mainland heightmap file contains the actual terrain heightmap (not the player map).
_git/artwork/maps/crownspire-player-map.png Ensures the Crownspire player map file contains the actual player map (not the heightmap).
_git/artwork/heightmaps/crownspire-heightmap.png Ensures the Crownspire heightmap file contains the actual terrain heightmap (not the player map).
_git/artwork/maps/frostfang-player-map.png Adds Frostfang player map asset in the standard maps directory.
_git/artwork/heightmaps/frostfang-heightmap.png Adds Frostfang terrain heightmap asset in the standard heightmaps directory.
_git/artwork/maps/sylvanreach-player-map.png Adds Sylvanreach player map asset in the standard maps directory.
_git/artwork/heightmaps/sylvanreach-heightmap.png Adds Sylvanreach terrain heightmap asset in the standard heightmaps directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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