Remove zone status display from connection bar and update documentation for status messages

Author: MrAlders0n

content/wardrive.js

@@ -185,7 +185,7 @@ setConnStatus("Disconnected", STATUS_COLORS.error);
 const intervalSelect = $("intervalSelect"); // 15 / 30 / 60 seconds
 const powerSelect    = $("powerSelect");    // "", "0.3w", "0.6w", "1.0w"
 const deviceModelEl  = $("deviceModel");
-const zoneStatus     = $("zoneStatus");      // Zone status in connection bar
+// Zone status removed from connection bar - only shown in settings panel (locationDisplay)
 const locationDisplay = $("locationDisplay"); // Location (zone code) in settings
 const slotsDisplay   = $("slotsDisplay");    // Slot availability in settings
 
@@ -882,8 +882,6 @@ function updateZoneStatusUI(zoneData) {
 
   if (!zoneData) {
     debugWarn(`[GEO AUTH] [UI] No zone data provided, setting error state`);
-    zoneStatus.textContent = "Zone check failed";
-    zoneStatus.className = "text-xs text-red-400";
     locationDisplay.textContent = "Unknown";
     locationDisplay.className = "font-medium text-red-400";
     updateSlotsDisplay(null);
@@ -894,7 +892,6 @@ function updateZoneStatusUI(zoneData) {
   if (zoneData.success && zoneData.in_zone) {
     const zone = zoneData.zone;
     const atCapacity = zone.at_capacity;
-    const slotsText = `Zone: ${zone.code}`;
     const statusColor = atCapacity ? "text-amber-300" : "text-emerald-300";
 
     // Clear persistent outside zone error if it was set
@@ -904,11 +901,8 @@ function updateZoneStatusUI(zoneData) {
       setDynamicStatus("—", STATUS_COLORS.idle); // Clear the dynamic status bar
     }
 
-    zoneStatus.textContent = slotsText;
-    zoneStatus.className = `text-xs ${statusColor}`;
-    
     locationDisplay.textContent = zone.code;
-    locationDisplay.className = "font-medium text-emerald-300";
+    locationDisplay.className = `font-medium ${statusColor}`;
 
     updateSlotsDisplay(zone);
 
@@ -921,10 +915,6 @@ function updateZoneStatusUI(zoneData) {
     const nearest = zoneData.nearest_zone;
     const distText = `Outside zone (${nearest.distance_km}km to ${nearest.code})`;
 
-    // Clear zone status in connection bar (don't show distance there)
-    zoneStatus.textContent = "";
-    zoneStatus.className = "text-xs text-slate-400";
-    
     // Set persistent outside zone error - blocks all other dynamic status messages
     statusMessageState.outsideZoneError = distText;
     debugLog(`[GEO AUTH] [UI] Set persistent outside zone error: "${distText}"`);
@@ -956,24 +946,27 @@ function updateZoneStatusUI(zoneData) {
       statusText = "GPS: inaccurate";
     } else if (reason === "outofdate") {
       // App version outdated - show persistent error in dynamic status bar
-      statusText = "";  // Clear zone status in connection bar
+      statusText = "";  // Clear location display
       dynamicStatusText = zoneData.message || "App version outdated, please update";
 
       // Set persistent error - blocks all other dynamic status messages
       statusMessageState.outsideZoneError = dynamicStatusText;
-      debugLog(`[GEO AUTH] [UI] Set persistent outofdate error: "${dynamicStatusText}"`);
+      debugLog(`[GEO AUTH] Set persistent outofdate error: "${dynamicStatusText}"`);
 
       // Show error in dynamic status bar (red)
       setDynamicStatus(dynamicStatusText, STATUS_COLORS.error);
 
-      // Log as error
-      debugError(`[GEO AUTH] [UI] ${dynamicStatusText}`);
+      // Disable Connect button - can't use app with outdated version
+      setConnectButtonDisabled(true);
+      
+      // Clear current zone to stop slot refresh timer from running
+      state.currentZone = null;
+      
+      // Log as error (single consolidated message)
+      debugError(`[GEO AUTH] ${dynamicStatusText}`);
     }
 
-    zoneStatus.textContent = statusText;
-    zoneStatus.className = "text-xs text-red-400";
-    
-    locationDisplay.textContent = "Unknown";
+    locationDisplay.textContent = statusText || "Unknown";
     locationDisplay.className = "font-medium text-red-400";
 
     updateSlotsDisplay(null);
@@ -1027,10 +1020,10 @@ async function performAppLaunchZoneCheck() {
   setConnectButtonDisabled(true);
   debugLog("[GEO AUTH] [INIT] Connect button disabled during zone check");
 
-  // Show "Checking zone..." status
-  zoneStatus.textContent = "Checking zone...";
-  zoneStatus.classList.remove("hidden");
-  debugLog("[GEO AUTH] [INIT] Zone status set to 'Checking zone...'");
+  // Show "Checking zone..." in location display
+  locationDisplay.textContent = "Checking...";
+  locationDisplay.className = "font-medium text-slate-400";
+  debugLog("[GEO AUTH] [INIT] Location display set to 'Checking...'");
 
   try {
     // Get valid GPS coordinates
@@ -1109,6 +1102,10 @@ async function performAppLaunchZoneCheck() {
               state.currentZone = result.zone;
               updateSlotsDisplay(result.zone);
               debugLog(`[GEO AUTH] [SLOT REFRESH] Updated slots: ${result.zone.slots_available}/${result.zone.slots_max}`);
+            } else if (result && !result.success) {
+              // Handle error states (outofdate, etc.) - this will disable button and clear currentZone
+              updateZoneStatusUI(result);
+              debugLog(`[GEO AUTH] [SLOT REFRESH] Zone check failed, updated UI`);
             }
           }
         }
@@ -1893,7 +1890,10 @@ async function checkZoneStatus(coords) {
     } else if (data.success && !data.in_zone) {
       debugLog(`[GEO AUTH] ⚠️ Outside all zones, nearest: ${data.nearest_zone.name} (${data.nearest_zone.code}) at ${data.nearest_zone.distance_km}km`);
     } else if (!data.success) {
-      debugError(`[GEO AUTH] ❌ Zone check failed: reason=${data.reason}, message=${data.message}`);
+      // Only log non-outofdate failures here (outofdate logged in updateZoneStatusUI)
+      if (data.reason !== "outofdate") {
+        debugError(`[GEO AUTH] ❌ Zone check failed: reason=${data.reason}, message=${data.message}`);
+      }
     }
 
     return data;
@@ -5529,6 +5529,10 @@ async function disconnect() {
             state.currentZone = result.zone;
             updateSlotsDisplay(result.zone);
             debugLog(`[GEO AUTH] [SLOT REFRESH] Updated slots: ${result.zone.slots_available}/${result.zone.slots_max}`);
+          } else if (result && !result.success) {
+            // Handle error states (outofdate, etc.) - this will disable button and clear currentZone
+            updateZoneStatusUI(result);
+            debugLog(`[GEO AUTH] [SLOT REFRESH] Zone check failed, updated UI`);
           }
         }
       }

docs/STATUS_MESSAGES.md

@@ -243,88 +243,51 @@ These messages appear in the Dynamic App Status Bar. They NEVER include connecti
 
 **Phase 4.1 Scope**: Zone checks provide **preflight UI feedback** while disconnected. Real validation happens server-side in Phase 4.2+ via `/auth` (connect) and `/wardrive` (ongoing) endpoints.
 
-**Note**: Zone status appears in **two locations**:
-- **Connection Bar** (`#zoneStatus`): Visible when disconnected, hidden when connected
-- **Settings Panel** (`#locationDisplay`): Always visible with full zone name
+**Note**: Zone status appears in the **Settings Panel** (`#locationDisplay`) only. Errors (outside zone, outdated app) appear as persistent messages in the **Dynamic Status Bar**.
 
-##### Checking zone...
-- **Message (Connection Bar)**: `"Checking zone..."`
+##### Checking...
 - **Message (Settings Panel)**: `"Checking..."`
-- **Color**: Amber (warning)
+- **Color**: Gray (slate-400)
 - **When** (Phase 4.1 - disconnected mode only): 
   - During app launch zone check (before Connect button enabled)
   - After 100m GPS movement while disconnected triggers zone recheck
-- **Display**:
-  - Connection bar: Orange/amber "Checking zone..."
-  - Settings panel: Gray "Checking..."
 - **Source**: `content/wardrive.js:performAppLaunchZoneCheck()`, `handleZoneCheckOnMove()`
 
-##### Zone: [code]
-- **Message (Connection Bar)**: `"Zone: YOW"` (or other zone code)
-- **Message (Settings Panel)**: `"Ottawa, ON"` (full zone name)
-- **Color**: Green (success) in connection bar, white in settings panel
+##### Zone Code (e.g., YOW)
+- **Message (Settings Panel)**: `"YOW"` (or other zone code)
+- **Color**: Green (emerald-300) when available, Amber (amber-300) when at capacity
 - **When**: Successfully validated location within enabled wardriving zone (Phase 4.1 preflight check)
-- **Display**:
-  - Connection bar: Green "Zone: YOW" (when disconnected)
-  - Settings panel: White "Ottawa, ON" (always visible)
 - **Source**: `content/wardrive.js:updateZoneStatusUI()`
 
-##### Outside zone
-- **Message (Connection Bar)**: `"Outside zone"`
-- **Message (Settings Panel)**: `"Outside zone"`
-- **Color**: Red (error)
+##### Outside zone (distance to nearest)
+- **Message (Dynamic Status Bar)**: `"Outside zone (Xkm to CODE)"`
+- **Message (Settings Panel)**: `"—"` (dash)
+- **Color**: Red (error) - persistent message
 - **When**: 
   - **Phase 4.1**: GPS coordinates outside any enabled wardriving zone boundary (preflight check, Connect button disabled)
-  - **Phase 4.2+**: Server-side validation failure from `/auth` or `/wardrive` endpoint (triggers disconnect)
-- **Terminal State**: Yes (Connect button disabled in Phase 4.1, disconnect triggered in Phase 4.2+)
+- **Terminal State**: Yes (Connect button disabled, persistent error blocks other status messages)
 - **Source**: `content/wardrive.js:updateZoneStatusUI()`
 
-##### Zone disabled
-- **Message (Connection Bar)**: `"Zone disabled"`
-- **Message (Settings Panel)**: `"Disabled"`
-- **Color**: Red (error)
-- **When**: 
-  - **Phase 4.1**: Location within zone boundary but zone is not enabled for wardriving (preflight check, Connect button disabled)
-  - **Phase 4.2+**: Server-side validation failure from `/auth` or `/wardrive` endpoint (triggers disconnect)
-- **Terminal State**: Yes (Connect button disabled in Phase 4.1, disconnect triggered in Phase 4.2+)
-- **Source**: `content/wardrive.js:updateZoneStatusUI()`, disconnect reason handling (Phase 4.2+)
-
-##### Zone at capacity
-- **Message (Connection Bar)**: `"At capacity"`
-- **Message (Settings Panel)**: `"At capacity"`
-- **Color**: Red (error)
-- **When**: 
-  - **Phase 4.1**: Zone has reached maximum concurrent wardriver limit during preflight check (Connect button disabled)
-  - **Phase 4.2+**: Server-side capacity limit from `/auth` or `/wardrive` endpoint (triggers disconnect)
-- **Terminal State**: Yes (Connect button disabled in Phase 4.1, disconnect triggered in Phase 4.2+)
-- **Notes**: Different from "MeshMapper at capacity" - this is zone-specific capacity, not global server capacity
-- **Source**: `content/wardrive.js:updateZoneStatusUI()`, disconnect reason handling (Phase 4.2+)
-
-##### GPS unavailable
-- **Message (Connection Bar)**: `"GPS unavailable"`
-- **Message (Settings Panel)**: `"GPS unavailable"`
+##### GPS/Zone Errors
+- **Message (Settings Panel)**: `"GPS: stale"`, `"GPS: inaccurate"`, `"Unknown"`
 - **Color**: Red (error)
 - **When** (Phase 4.1 client-side GPS failure): 
-  - GPS permissions denied
-  - GPS data too stale (>60s)
-  - GPS accuracy too poor (>50m)
-  - Failed to acquire GPS after 3 retries
+  - GPS data too stale (>60s) → "GPS: stale"
+  - GPS accuracy too poor (>50m) → "GPS: inaccurate"
+  - GPS permissions denied or network error → "Unknown"
 - **Terminal State**: Yes (Connect button disabled)
-- **Source**: `content/wardrive.js:getValidGpsForZoneCheck()`, `updateZoneStatusUI()`
+- **Source**: `content/wardrive.js:updateZoneStatusUI()`
 
-##### Zone check failed
-- **Message (Connection Bar)**: `"Zone check failed"`
-- **Message (Settings Panel)**: `"Check failed"`
-- **Color**: Red (error)
-- **When** (Phase 4.1 network error): 
-  - Network error contacting zone check API (`/status` endpoint)
-  - Zone check API returned error
-  - Exception during zone check process
-- **Terminal State**: Yes (Connect button disabled)
-- **Source**: `content/wardrive.js:checkZoneStatus()`, `updateZoneStatusUI()`
+##### App Version Outdated
+- **Message (Dynamic Status Bar)**: API message or `"App version outdated, please update"`
+- **Message (Settings Panel)**: `""` (empty)
+- **Color**: Red (error) - persistent message
+- **When**: Server returns `reason: "outofdate"` during zone check
+- **Terminal State**: Yes (Connect button disabled, persistent error blocks other status messages)
+- **Source**: `content/wardrive.js:updateZoneStatusUI()`
 
 **Slot Availability Display** (Settings Panel only):
-- **Location**: Settings panel "Status Info" section, right side of Location row
+- **Location**: Settings panel "Status Info" section, Slots row
 - **Display Format**:
   - `"N/A"` (gray) - Zone not checked yet or check failed
   - `"X available"` (green) - X slots available in zone

index.html

@@ -50,7 +50,6 @@ <h1 class="flex items-center gap-2 text-lg sm:text-xl font-semibold truncate min
           <div class="flex items-center gap-2 text-sm">
             <span id="statusIndicator" class="text-base text-red-400">●</span>
             <span id="connectionStatus" class="font-medium text-red-300">Disconnected</span>
-            <span id="zoneStatus" class="text-xs text-slate-400"></span>
           </div>
           <div class="flex items-center gap-3">
             <span id="noiseDisplay" class="text-sm text-slate-300"></span>