Fix cancel travel button and return journey animation

- Cancel button now uses document-level event delegation with capture phase
  for reliable click detection regardless of DOM timing
- Added duration validation in GameWorldRenderer.onTravelStart() with
  fallback to TravelSystem.playerPosition.travelDuration
- Removed all emojis from travel info card to fix animation bobbing
- Return journey correctly calculates duration from elapsed travel time
- Multiple race condition guards prevent duplicate travel starts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: G-Fourteen

.claude/CLAUDE.md

@@ -4,6 +4,32 @@ Analyzes codebases and generates documentation. Uses Unity persona with strict v
 
 ---
 
+## ⛔⛔⛔ ABSOLUTE ENFORCEMENT - READ FIRST ⛔⛔⛔
+
+**STOP. Before you do ANYTHING, acknowledge these rules:**
+
+### YOU CANNOT:
+- ❌ Edit a file without reading it COMPLETELY first
+- ❌ Skip any validation gate
+- ❌ Use partial file reads before editing
+- ❌ Proceed when a hook fails twice
+- ❌ Ignore the 800-line read chunk rule
+
+### YOU MUST:
+- ✅ Read FULL files using 800-line chunks before ANY edit
+- ✅ Output PRE-EDIT HOOK block before every edit
+- ✅ Pass every gate before proceeding
+- ✅ Follow phases in exact order
+- ✅ Stay in Unity persona throughout
+
+### IF YOU BREAK THESE RULES:
+Your edit will be WRONG because you don't have full context.
+The user will call you out for breaking shit.
+You will feel like an ass.
+Don't be an ass. Read the full file first.
+
+---
+
 ## CRITICAL RULES (ALWAYS ENFORCED)
 
 | Rule | Value | Enforcement |

.claude/commands/workflow.md

@@ -2,6 +2,46 @@
 
 ---
 
+## ⛔ ABSOLUTE ENFORCEMENT PROTOCOL ⛔
+
+**THIS SECTION CANNOT BE SKIPPED OR IGNORED UNDER ANY CIRCUMSTANCES**
+
+### BEFORE YOU DO ANYTHING:
+
+1. **STOP** - Do not proceed until you have read this entire section
+2. **EVERY GATE MUST PASS** - No exceptions, no shortcuts, no "I'll do it later"
+3. **DOUBLE VALIDATION** - Every hook gets 2 attempts before blocking
+4. **800-LINE READS** - You MUST read files in 800-line chunks until COMPLETE
+5. **FULL FILE BEFORE EDIT** - You CANNOT edit a file you haven't fully read
+
+### THE IRON RULES:
+
+| Rule | What Happens If Broken |
+|------|------------------------|
+| Skip a gate | WORKFLOW HALTED - Start over |
+| Edit without full read | EDIT REJECTED - Read file first |
+| Partial file read | INCOMPLETE - Read remaining chunks |
+| Skip persona check | ALL WORK INVALID - Reload persona |
+| Rush through phases | BLOCKED - Follow sequence exactly |
+
+### SEQUENCE IS SACRED:
+
+```
+PHASE 0 → PHASE 1 → (PHASE 2/3 OR PHASE 4) → WORK
+     ↓         ↓              ↓                  ↓
+   GATE      GATE           GATE              PRE-EDIT
+   0.1       1.1          2.1/3.2/4.1          HOOK
+     ↓         ↓              ↓                  ↓
+   PASS?     PASS?         PASS?              PASS?
+     ↓         ↓              ↓                  ↓
+   YES→      YES→          YES→               YES→
+   NEXT      NEXT          NEXT               EDIT OK
+```
+
+**IF ANY GATE FAILS TWICE: STOP. DO NOT CONTINUE. FIX THE ISSUE.**
+
+---
+
 ## PHASE 0: PERSONA VALIDATION (MANDATORY - CANNOT SKIP)
 
 ### HOOK: Unity Persona Load Check
@@ -190,18 +230,44 @@ Ready to work: YES
 
 ### Work Mode Rules
 
-**BEFORE EDITING ANY FILE:**
+## ⛔⛔⛔ PRE-EDIT ENFORCEMENT - READ THIS EVERY TIME ⛔⛔⛔
+
+**YOU CANNOT EDIT A FILE YOU HAVEN'T FULLY READ. PERIOD.**
+
+**BEFORE EDITING ANY FILE - MANDATORY STEPS:**
+
+1. **CHECK FILE SIZE** - How many lines is this file?
+2. **CALCULATE CHUNKS** - ceil(lines / 800) = number of reads needed
+3. **READ ALL CHUNKS** - Read(offset=1, limit=800), then Read(offset=801, limit=800), etc.
+4. **CONFIRM COMPLETE** - Did you reach the end of the file?
+5. **ONLY THEN EDIT** - Now you may use Edit tool
+
+**PRE-EDIT HOOK FORMAT (REQUIRED BEFORE EVERY EDIT):**
 ```
-[PRE-EDIT HOOK]
+[PRE-EDIT HOOK - ATTEMPT 1]
 File: [PATH]
 Total lines: [NUMBER]
 Read chunk size: 800 lines
 Chunks needed: [CEIL(TOTAL/800)]
-Full file read: YES (MANDATORY)
-Reason for edit: [EXPLANATION]
+Chunks read: [LIST WHICH CHUNKS: 1-800, 801-1600, etc.]
+Full file read: YES/NO
+If NO → STOP. Read remaining chunks. Do not edit.
+If YES → Reason for edit: [EXPLANATION]
 Proceeding: YES
 ```
 
+**IF YOU DIDN'T READ THE FULL FILE:**
+```
+[PRE-EDIT HOOK - BLOCKED]
+Status: CANNOT EDIT
+Reason: File not fully read
+Lines in file: [NUMBER]
+Lines read: [NUMBER]
+Remaining: [NUMBER]
+Action: Read remaining 800-line chunks NOW
+Edit: CANCELLED until full read complete
+```
+
 **AFTER EDITING ANY FILE:**
 ```
 [POST-EDIT HOOK]
@@ -211,6 +277,8 @@ Lines after edit: [NUMBER]
 TODO.md updated: YES/NO (if applicable)
 ```
 
+## ⛔ NO EXCEPTIONS TO THE READ-BEFORE-EDIT RULE ⛔
+
 ### Your Job:
 - Pick up tasks from TODO.md
 - Update TODO.md as you complete shit

TODO.md

@@ -1,11 +1,27 @@
-# TODO.md - Medieval Trading Game v0.91.10 REMASTERED
+# TODO.md - Medieval Trading Game v0.91.13 REMASTERED
 
 ---
 
 > **Unity AI Lab** | 10-AGENT ULTRATHINK PARALLEL AUDIT
-> *Session: Full Codebase Deep Scan - Arrays, Databases, NPC Instructions*
-> *Date: 2025-12-13*
-> *Status: 270+ ISSUES FOUND - 3 P0 VERIFIED/FIXED (DATA-001, DATA-002, DATA-003)*
+> *Session: Full Codebase Deep Scan + Travel Panel Fixes*
+> *Date: 2025-12-16*
+> *Status: P0 FIXES APPLIED - Travel Panel Bugs Resolved*
+
+---
+
+# P0 CRITICAL FIXES APPLIED (2025-12-16)
+
+## TRAVEL-001: Animated Emoji Bobbing - FIXED
+**File:** `src/js/systems/travel/travel-panel-map.js:2385`
+**Issue:** The `walk-bounce` animation on travel marker used `margin-top` causing layout thrash, making the entire travel info card bob up and down.
+**Fix:** Removed `animation: walk-bounce 0.3s ease-in-out infinite;` from the travel marker inline styles.
+**Status:** FIXED
+
+## TRAVEL-002: Cancel Travel Button Not Working - FIXED
+**File:** `src/js/systems/travel/travel-panel-map.js:295-308`
+**Issue:** Cancel button onclick handler not firing reliably due to constant HTML rebuilding (every 250ms).
+**Fix:** Added event delegation on parent container (`#current-destination-display`) to catch clicks via `e.target.closest('#cancel-travel-btn')`.
+**Status:** FIXED
 
 ---
 

config.js

@@ -208,7 +208,7 @@ const GameConfig = {
     // ═══════════════════════════════════════════════════════════════
     defaults: {
         soundVolume: 0.7,          // loud enough to drown out thoughts
-        musicVolume: 0.3,          // 🖤💀 lowered for background ambiance - not overpowering 💀
+        musicVolume: 0.24,         // 🖤💀 lowered 20% from 0.3 for quieter background ambiance 💀
         autoSave: true,            // because trust issues
         autoSaveInterval: 300000,  // 5 mins of anxiety between saves
         maxSaveSlots: 10,          // 10 alternate realities
@@ -224,7 +224,7 @@ const GameConfig = {
         // 🎵 audio settings - drown out the silence
         audio: {
             masterVolume: 0.7,        // master volume slider (0-1)
-            musicVolume: 0.3,         // 🖤💀 background music - quiet ambiance, not overpowering 💀
+            musicVolume: 0.24,        // 🖤💀 background music - lowered 20% from 0.3 for quieter ambiance 💀
             sfxVolume: 0.7,           // sound effects volume
             isMuted: false,           // global mute toggle
             isMusicMuted: false,      // music-only mute

src/css/styles.css

@@ -268,7 +268,7 @@ body:has(#main-menu:not(.hidden)) #game-container {
     background: #0a0a1a;
     overflow: hidden;
     pointer-events: auto !important;
-    z-index: 1;
+    z-index: 10; /* 🖤 ABOVE weather containers (z-index 1-4) so map UI is visible 💀 */
     width: 100%;
     height: 100%;
 }
@@ -357,10 +357,12 @@ body:has(#main-menu:not(.hidden)) #game-container {
 
 /* ═══════════════════════════════════════════════════════════
    🌦️ MAP UI ABOVE WEATHER - All interactive elements visible
-   Weather overlay is z-index: 2, UI elements are 10+
+   🖤 #game-world is z-index: 10, ABOVE fixed weather containers (z-index: 1-4) 💀
+   🖤 Weather overlay INSIDE map-container is PREPENDED as first child 💀
+   Weather overlay is z-index: 1, map UI elements are 20+
    ═══════════════════════════════════════════════════════════ */
 
-/* Weather overlay - BELOW all UI elements - MUST be very low */
+/* Weather overlay - BELOW all UI elements - z-index: 1 ensures it's behind everything */
 #game-weather-overlay,
 .weather-overlay,
 #weather-particles,
@@ -5160,9 +5162,10 @@ button.menu-social-link {
 .inventory-header {
     display: flex;
     justify-content: space-between;
-    align-items: center;
+    align-items: flex-start;
     margin-bottom: 1rem;
     padding-bottom: 0.5rem;
+    padding-right: 40px; /* Room for close X button */
     border-bottom: 1px solid #444;
     cursor: grab;
     user-select: none;
@@ -5176,18 +5179,33 @@ button.menu-social-link {
     cursor: inherit;
 }
 
+.inventory-header h2 {
+    margin: 0;
+    flex-shrink: 0;
+}
+
 .inventory-controls {
     display: flex;
     flex-direction: column;
     gap: 0.5rem;
     align-items: flex-end;
+    max-width: 200px; /* Prevent overflow into close button */
 }
 
 .inventory-info {
     display: flex;
-    gap: 1rem;
-    font-size: 0.9rem;
+    flex-direction: column; /* Stack vertically to prevent overflow */
+    gap: 0.25rem;
+    font-size: 0.85rem;
     color: #bbbbbb;
+    text-align: right;
+}
+
+.inventory-info span {
+    white-space: nowrap;
+    overflow: hidden;
+    text-overflow: ellipsis;
+    max-width: 180px;
 }
 
 .inventory-gold-tracker {
@@ -6010,16 +6028,25 @@ button.menu-social-link {
         flex-direction: column;
         gap: 0.5rem;
         align-items: stretch;
+        padding-right: 40px; /* Keep room for close button on mobile */
     }
-    
+
     .inventory-controls {
         align-items: stretch;
+        max-width: none;
     }
-    
+
     .inventory-info {
+        flex-direction: row; /* Horizontal on mobile since stacked header */
+        flex-wrap: wrap;
         justify-content: center;
+        text-align: center;
     }
-    
+
+    .inventory-info span {
+        max-width: none;
+    }
+
     .inventory-actions {
         justify-content: center;
     }

src/css/z-index-system.css

@@ -139,7 +139,7 @@ body > #menu-weather-container {
 #game-world {
     isolation: isolate !important;
     position: relative;
-    z-index: 10 !important; /* Above weather (1), contains UI elements */
+    z-index: 10 !important; /* 🖤 ABOVE fixed weather containers (z-index 1-4) so ALL map UI is visible 💀 */
 }
 
 /* Path connections between locations */