Expedition: readable map art, stage counter math, place-name title - #230
Merged
Conversation
Addressing three playtest reports on the expedition map screen. ## 1. Background image too high contrast The flux parchment-map background was rendering at full opacity behind nodes + labels. Forest patches drowned the kind labels — the CSS parchment baseline from PR #221 had been more legible. - Drop the flux <image> to 55% opacity so the warm parchment underneath bleeds through. - Add a cream wash rect (#e8d5a8 @ 28% alpha) on top of the image to flatten the dynamic range into a readable backdrop. - Reduce the noise filter opacity from 40% → 35% over art so the grain doesn't compound the murk. - Paint kind labels with paintOrder="stroke" + a thin cream (#f3e6c3) outline so the text reads on any background patch without darkening the art. ## 2. "0 of 38 nodes cleared" doesn't match what the player does Counting every node in the graph (lanes × depth, minus start/boss) is ~36-40 for a typical map but the player only ever makes ~14 picks (one per depth column plus the boss). Switch the counter to \`view.map.depth + 1\` and call them "stages" — matches the cadence of "how many picks have I made vs. the total" instead of "how many lanes have I traveled." ## 3. Expedition needs a title The header read "Expedition #N" — fine for an id but bland. Add EXPEDITION_MAP_TITLES in ai.ts mapping each map_variant to a proper place-name ("The Misty Mountains", "The Wraith Fens", etc.). Worker GET response now includes \`map_title\`; header renders it as the big display-font subhead under "Expedition #N" so a run reads as "Expedition #10 — The Rolling Hills." Same variant selection is already deterministic by seed (PR #224), so the title stays stable across reloads for the same expedition. Verified the CSS-only fallback in the local preview. Flux art path is disabled in local dev (ENVIRONMENT=local turns artTarget.disabled true); overlay layers are conservative so the painted variant will land at least as readable as the unpainted one. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Three playtest reports rolled into one PR:
Test plan
🤖 Generated with Claude Code