Skip to content

Cut jargon from the map and timelapse wording - #131

Merged
ryanbarlow97 merged 2 commits into
mainfrom
map-wording
Oct 8, 2026
Merged

ryanbarlow97 merged 2 commits into
mainfrom
map-wording

Conversation

@ryanbarlow97

Copy link
Copy Markdown
Contributor

Part of a site-wide pass to remove jargon and clutter from front-end text. The wording was reviewed with Astra (Codex gpt-6-astra). See also #130 for the staff panel.

What changes

  • Map access and missing days: plain messages throughout.
    • "Staff only" replaces the LuckPerms node and "meta syncs" text.
    • "Sign in to view this map" now links to Profile, the page with the code form (previously /character).
    • The "showing today's would be a lie about the past" and "file behind this map mode is not in its snapshot" paragraphs are gone.
    • Formatted dates replace raw YYYY-MM-DD.
    • "Back to Realms" replaces "Back to the nation map".
  • Map panels: removed repeated labels.
    • The "World map" eyebrow is gone.
    • The tier appears once (it was shown three times).
    • "Kingdom of X" became the type alone; the realm link sits underneath.
    • "Earlier chapters" is now the archive button itself.
    • "Character not yet known" → "Unknown".
  • War planning toolbar:
    • "Paint mode — on" → "War planning".
    • "Export/Import JSON" → "Download plan" / "Open plan".
    • "Plate" → "Background".
    • The swatch tooltips no longer show hex codes.
    • The help line is shorter but keeps the middle-drag and scroll hints.
  • Timelapse:
    • "chronicle" becomes "timelapse" or "map history".
    • Snapshot, geometry and source wording is removed.
    • The build counters and memory estimate are gone.
    • Day-viewer notices are short.
    • "Nation names aren't available for this map." stays visible, not only as a tooltip.
  • Ledger charts:
    • The pseudo-code is gone.
    • The plain "Daily wealth change" note keeps the actual-versus-forecast distinction.
    • New captions: "Server-wide money supply (separate scale)" and "inflation adjustment (this faction)".
    • "SimpleFactions has not started posting snapshots" → "No economy data yet."
  • Staff-only tools (chronicle console, title editor, precedent, code checker): implementation asides are removed. Accountability notes stay.

Checks

  • tsc --noEmit passes.
  • vitest run: all 1975 tests pass.

🤖 Generated with Claude Code

ryanbarlow97 and others added 2 commits October 8, 2026 17:53
Plain access and missing-day messages, drop repeated tier and realm labels,
call paint mode War planning, and remove snapshot, geometry and build-counter
details from the timelapse and ledger panels.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Restore the forecast-versus-actual note, the visible nation-names notice,
the pan and zoom hint and the code checker's note; say 'map history' in
load errors and tighten the ledger legend.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 20 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: df1b7d04-1bc7-449c-af7d-f87881c97fe6
📥 Commits

Reviewing files that changed from the base of the PR and between ee62af4 and bff6f77.

📒 Files selected for processing (32)
  • frontend/app/components/MapViewer.tsx
  • frontend/app/components/chronicle/ChronicleDayViewer.tsx
  • frontend/app/components/chronicle/ChroniclePanels.test.tsx
  • frontend/app/components/chronicle/ChroniclePanels.tsx
  • frontend/app/components/chronicle/ChronicleStaffConsole.test.tsx
  • frontend/app/components/chronicle/ChronicleStaffConsole.tsx
  • frontend/app/components/chronicle/ChronicleStudio.tsx
  • frontend/app/components/chronicle/InvalidChronicleDay.tsx
  • frontend/app/components/chronicle/LedgerChartsPanel.tsx
  • frontend/app/components/chronicle/chronicleLayers.ts
  • frontend/app/components/map/MapAccessGate.tsx
  • frontend/app/components/map/MapArchiveMenu.tsx
  • frontend/app/components/map/PaintToolbar.tsx
  • frontend/app/components/map/editor/EditorDisabledGate.tsx
  • frontend/app/components/map/editor/EditorEntryGate.tsx
  • frontend/app/components/map/editor/EditorSaveBar.tsx
  • frontend/app/components/map/editor/MapEditorCanvas.tsx
  • frontend/app/components/map/regionInfo.ts
  • frontend/app/components/map/shell/GuildPanel.tsx
  • frontend/app/components/map/shell/MapPlaque.tsx
  • frontend/app/components/map/shell/RealmPanel.tsx
  • frontend/app/components/map/types.ts
  • frontend/app/components/precedent/PrecedentCaseModal.tsx
  • frontend/app/components/precedent/PrecedentSearchPanel.tsx
  • frontend/app/components/site/CodeInspector.tsx
  • frontend/app/components/site/DevLandingPage.tsx
  • frontend/app/hooks/useMapHover.ts
  • frontend/app/hooks/useRegionHover.ts
  • frontend/app/lib/map/archiveMaps.ts
  • frontend/app/lib/map/chronicleBuild.test.ts
  • frontend/app/lib/map/chronicleBuild.ts
  • frontend/app/precedent/page.tsx
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ryanbarlow97
ryanbarlow97 merged commit fb4d6ce into main Oct 8, 2026
6 checks passed
@ryanbarlow97
ryanbarlow97 deleted the map-wording branch October 8, 2026 18:20

This branch was successfully deployed

1 active deployment
preview — bff6f77b Deployed Oct 8, 2026 by ryanbarlow97 via deploy #33
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