Fix disabled ship UX and improve input handling

- Block burn/overload/weak-gravity input for disabled ships
- Skip disabled and landed ships in ordnance phase selection
- Tab key cycles ships in ordnance and combat phases too
- Ordnance buttons disabled when selected ship lacks cargo capacity
  (mine needs 10, torpedo/nuke need 20 mass)
- Button opacity indicates availability

138 tests passing.

Author: rgilks

src/client/input.ts

@@ -162,7 +162,7 @@ export class InputHandler {
     // Check if clicking a burn or overload direction arrow
     if (this.planningState.selectedShipId) {
       const ship = this.gameState.ships.find(s => s.id === this.planningState.selectedShipId);
-      if (ship && ship.fuel > 0) {
+      if (ship && ship.fuel > 0 && ship.damage.disabledTurns === 0) {
         const currentBurn = this.planningState.burns.get(ship.id) ?? null;
         const predDest = ship.landed
           ? computeCourse(ship, null, this.map).path[0] // launch hex
@@ -252,9 +252,10 @@ export class InputHandler {
       }
     }
 
-    // Check if clicking on own ship to select it
+    // Check if clicking on own ship to select it (skip disabled/landed ships for ordnance)
     for (const ship of this.gameState.ships) {
       if (ship.owner !== this.playerId || ship.destroyed) continue;
+      if (ship.damage.disabledTurns > 0 || ship.landed) continue;
       if (hexEqual(clickHex, ship.position)) {
         this.planningState.selectedShipId = ship.id;
         this.planningState.torpedoAccel = null;

src/client/main.ts

@@ -61,7 +61,8 @@ class GameClient {
 
     // Keyboard shortcuts
     document.addEventListener('keydown', (e) => {
-      if (e.key === 'Tab' && this.state === 'playing_astrogation' && this.gameState) {
+      if (e.key === 'Tab' && this.gameState &&
+          (this.state === 'playing_astrogation' || this.state === 'playing_ordnance' || this.state === 'playing_combat')) {
         e.preventDefault();
         this.cycleShip(e.shiftKey ? -1 : 1);
       } else if (e.key === 'Escape') {

src/client/ui.ts

@@ -137,14 +137,27 @@ export class UIManager {
     const confirmBtn = document.getElementById('confirmBtn')!;
     confirmBtn.style.display = isMyTurn && phase === 'astrogation' ? 'inline-block' : 'none';
 
-    const launchMineBtn = document.getElementById('launchMineBtn')!;
-    const launchTorpedoBtn = document.getElementById('launchTorpedoBtn')!;
-    const launchNukeBtn = document.getElementById('launchNukeBtn')!;
+    const launchMineBtn = document.getElementById('launchMineBtn')! as HTMLButtonElement;
+    const launchTorpedoBtn = document.getElementById('launchTorpedoBtn')! as HTMLButtonElement;
+    const launchNukeBtn = document.getElementById('launchNukeBtn')! as HTMLButtonElement;
     const skipOrdnanceBtn = document.getElementById('skipOrdnanceBtn')!;
-    launchMineBtn.style.display = isMyTurn && phase === 'ordnance' ? 'inline-block' : 'none';
-    launchTorpedoBtn.style.display = isMyTurn && phase === 'ordnance' ? 'inline-block' : 'none';
-    launchNukeBtn.style.display = isMyTurn && phase === 'ordnance' ? 'inline-block' : 'none';
-    skipOrdnanceBtn.style.display = isMyTurn && phase === 'ordnance' ? 'inline-block' : 'none';
+    const showOrd = isMyTurn && phase === 'ordnance';
+    launchMineBtn.style.display = showOrd ? 'inline-block' : 'none';
+    launchTorpedoBtn.style.display = showOrd ? 'inline-block' : 'none';
+    launchNukeBtn.style.display = showOrd ? 'inline-block' : 'none';
+    skipOrdnanceBtn.style.display = showOrd ? 'inline-block' : 'none';
+    // Disable buttons based on cargo capacity
+    if (showOrd) {
+      const canMine = cargoFree >= 10; // mine mass
+      const canTorpedo = cargoFree >= 20; // torpedo mass
+      const canNuke = cargoFree >= 20; // nuke mass
+      launchMineBtn.disabled = !canMine;
+      launchTorpedoBtn.disabled = !canTorpedo;
+      launchNukeBtn.disabled = !canNuke;
+      launchMineBtn.style.opacity = canMine ? '1' : '0.4';
+      launchTorpedoBtn.style.opacity = canTorpedo ? '1' : '0.4';
+      launchNukeBtn.style.opacity = canNuke ? '1' : '0.4';
+    }
 
     const skipCombatBtn = document.getElementById('skipCombatBtn')!;
     skipCombatBtn.style.display = isMyTurn && phase === 'combat' ? 'inline-block' : 'none';