ok

src/entities/towers/Tower.js

@@ -134,6 +134,7 @@ export class Tower {
     this.ring = ring;
     this.hoverOutline = outline;
     this.levelRing = null;
+    this.levelRings = [];
 
     if (this.isElectric) {
       this.applyVisualLevel();
@@ -193,7 +194,17 @@ export class Tower {
   }
 
   applyVisualLevel() {
+    // Let the kind update its visuals first (head position, colors, base ring)
     this.kind.applyVisualLevel(this);
+
+    // Clean up any legacy/orphan base rings left from earlier versions
+    this._cleanupOrphanLevelRings();
+
+    // Then ensure a consistent number of visible rings across all towers:
+    // Level 1 => 0 rings; Level 2 => 1 ring; Level 3 => 2 rings, etc.
+    // Most kinds add a single base ring for level >= 2. We add additional
+    // rings above that so the total equals (level - 1).
+    this._updateAdditionalLevelRings();
   }
 
   tryFire(dt, enemies, projectiles, projectileSpeed) {
@@ -280,6 +291,97 @@ export class Tower {
       if (this.levelRing.material?.dispose) this.levelRing.material.dispose();
       this.levelRing = null;
     }
+    if (this.levelRings && this.levelRings.length) {
+      for (const ring of this.levelRings) {
+        this.scene.remove(ring);
+        ring.geometry?.dispose?.();
+        ring.material?.dispose?.();
+      }
+      this.levelRings.length = 0;
+    }
     this.scene.remove(this.mesh);
   }
-}
+
+  _updateAdditionalLevelRings() {
+    // Clean up any previously created extra rings
+    if (this.levelRings && this.levelRings.length) {
+      for (const ring of this.levelRings) {
+        this.scene.remove(ring);
+        ring.geometry?.dispose?.();
+        ring.material?.dispose?.();
+      }
+      this.levelRings.length = 0;
+    }
+
+    // How many rings should be visible in total
+    const totalRings = Math.max(0, (this.level || 1) - 1);
+    if (totalRings <= 1) {
+      // Kind likely already created the first ring if level >= 2; nothing else to do
+      return;
+    }
+
+    // Additional rings we should add above the kind's base ring
+    const extraRings = totalRings - 1;
+
+    // Placement parameters
+    const baseY = (this.headTopY ?? 0.8) + 0.02; // matches kind base ring offset
+    const spacing = 0.1; // increased vertical spacing between rings
+
+    // Choose a color consistent with the tower type
+    let color = 0x3aa6ff; // default blue
+    if (this.type === "slow") color = 0xff8fc2;
+    else if (this.type === "sniper") color = 0x00ffff;
+    // electric uses default blue
+
+    for (let i = 0; i < extraRings; i++) {
+      const geom = new THREE.TorusGeometry(0.45, 0.035, 8, 24);
+      const mat = new THREE.MeshStandardMaterial({
+        color,
+        emissive: 0xe01a6b,
+        emissiveIntensity: 0.55,
+        metalness: 0.3,
+        roughness: 0.45,
+      });
+      const ring = new THREE.Mesh(geom, mat);
+      ring.castShadow = false;
+      ring.receiveShadow = false;
+      ring.rotation.x = Math.PI / 2;
+      ring.name = "tower_level_ring_extra";
+
+      // Stack above the base ring with clear vertical separation
+      const y = baseY + (i + 1) * spacing;
+      ring.position.set(this.mesh.position.x, y, this.mesh.position.z);
+
+      this.scene.add(ring);
+      this.levelRings.push(ring);
+    }
+  }
+
+  _cleanupOrphanLevelRings() {
+    const scene = this.scene;
+    if (!scene || !scene.children) return;
+
+    const px = this.mesh?.position?.x ?? this.position?.x ?? 0;
+    const pz = this.mesh?.position?.z ?? this.position?.z ?? 0;
+    const threshold = 0.1; // only remove rings very near this tower
+
+    // Collect candidates first to avoid mutating while iterating
+    const toRemove = [];
+    for (const obj of scene.children) {
+      if (!obj || obj === this.levelRing) continue;
+      if (obj.name !== 'tower_level_ring') continue;
+      const ox = obj.position?.x ?? 0;
+      const oz = obj.position?.z ?? 0;
+      const dx = ox - px;
+      const dz = oz - pz;
+      if (dx * dx + dz * dz <= threshold * threshold) {
+        toRemove.push(obj);
+      }
+    }
+    for (const obj of toRemove) {
+      scene.remove(obj);
+      obj.geometry?.dispose?.();
+      obj.material?.dispose?.();
+    }
+  }
+}

src/entities/towers/kinds/basic.js

@@ -75,8 +75,7 @@ export default {
       tower.levelRing = null;
     }
 
-    const extraRaw = Math.max(0, (lvl - 1) * VISUAL_TOP_INCREMENT);
-    const visualExtra = Math.min(VISUAL_TOP_CAP, extraRaw);
+    // No height increment on level gain — only rings should reflect level
 
     if (lvl <= 1) {
       if (baseMat) {
@@ -85,8 +84,7 @@ export default {
         baseMat.emissiveIntensity = 0.0;
       }
 
-      head.scale.set(1, 1, 1);
-      head.position.y = 0.65 + visualExtra;
+      // Keep original head height/position; recompute top using current head position
       tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.4;
 
       headMat.color?.set?.(0x90caf9);
@@ -99,9 +97,8 @@ export default {
         baseMat.emissiveIntensity = 0.08;
       }
 
-      head.scale.set(1, 2, 1);
-      head.position.y = 0.65 + 0.4 + visualExtra;
-      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.8;
+      // Keep original head height/position; recompute top using current head position
+      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.4;
 
       headMat.color?.set?.(0xa5d6ff);
       headMat.emissive?.set?.(0x9a135a);
@@ -132,4 +129,4 @@ export default {
       tower.scene.add(ring);
     }
   },
-};
+};

src/entities/towers/kinds/electric.js

@@ -60,6 +60,8 @@ export default {
     tower.headMesh = head;
     tower.head = head;
     tower.headTopY = tower.mesh.position.y + head.position.y + 0.45;
+    // Remember base head Y so applyVisualLevel can position absolutely per level
+    tower._electricBaseHeadY = head.position.y;
 
     tower.trackedTargets = new Map();
     tower.arcPool = [];
@@ -360,12 +362,20 @@ export default {
     const head = tower.headMesh;
     if (!head) return;
 
-    const liftRaw = Math.max(0, (lvl - 1) * ELECTRIC_BALL_LIFT_PER_LEVEL);
-    const lift = Math.min(ELECTRIC_BALL_LIFT_CAP, liftRaw);
-    head.position.y = head.position.y + lift;
+    // Do not raise the head on level gain — height remains constant
+    const baseY = typeof tower._electricBaseHeadY === 'number' ? tower._electricBaseHeadY : head.position.y;
+    head.position.y = baseY;
     tower.headTopY =
       (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.45;
 
+    // Remove any existing base level ring before creating a new one
+    if (tower.levelRing) {
+      tower.scene.remove(tower.levelRing);
+      tower.levelRing.geometry?.dispose?.();
+      if (tower.levelRing.material?.dispose) tower.levelRing.material.dispose();
+      tower.levelRing = null;
+    }
+
     if (lvl > 1) {
       head.material.color?.set?.(0xa5d6ff);
       head.material.emissive?.set?.(0x9a135a);
@@ -430,4 +440,4 @@ export default {
       tower.arcPool.length = 0;
     }
   },
-};
+};

src/entities/towers/kinds/slow.js

@@ -83,8 +83,7 @@ export default {
       tower.levelRing = null;
     }
 
-    const extraRaw = Math.max(0, (lvl - 1) * VISUAL_TOP_INCREMENT);
-    const visualExtra = Math.min(VISUAL_TOP_CAP, extraRaw);
+    // No height increment on level gain — only rings should reflect level
 
     if (lvl <= 1) {
       if (baseMat) {
@@ -93,10 +92,8 @@ export default {
         baseMat.emissiveIntensity = 0.0;
       }
 
-      head.scale.set(1, 1, 1);
-      head.position.y = 0.8 + visualExtra;
-      tower.headTopY =
-        (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.55;
+      // Keep original head height/position; recompute top using current head position
+      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.4;
 
       headMat.color?.set?.(0xffb6c1);
       headMat.emissive?.set?.(0x4a0a2a);
@@ -108,9 +105,8 @@ export default {
         baseMat.emissiveIntensity = 0.08;
       }
 
-      head.scale.set(1.1, 1.15, 1.1);
-      head.position.y = 0.9 + visualExtra;
-      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.6;
+      // Keep original head height/position; recompute top using current head position
+      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.4;
 
       headMat.color?.set?.(0xffc6d9);
       headMat.emissive?.set?.(0x9a135a);
@@ -141,4 +137,4 @@ export default {
       tower.scene.add(ring);
     }
   },
-};
+};

src/entities/towers/kinds/sniper.js

@@ -171,9 +171,7 @@ export default {
         baseMat.roughness = 0.35;
       }
 
-      // Sniper cone head baseline
-      head.scale.set(1, 1, 1);
-      head.position.y = 0.95 + visualExtra;
+      // Keep original head height/position; recompute top using current head position
       tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.55;
 
       headMat.color?.set?.(0xb0bec5);
@@ -191,10 +189,8 @@ export default {
         baseMat.roughness = 0.3;
       }
 
-      // Slightly larger cone; raise by visualExtra
-      head.scale.set(1.1, 1.2, 1.1);
-      head.position.y = 0.95 + visualExtra;
-      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.6;
+      // Keep original head height/position; recompute top using current head position
+      tower.headTopY = (tower.mesh?.position.y ?? 0.25) + head.position.y + 0.55;
 
       headMat.color?.set?.(0x90a4ae);
       headMat.emissive?.set?.(0x550000);
@@ -228,7 +224,7 @@ export default {
       tower.scene.add(ring);
     }
 
-    // Update headTopY after position changes
+    // Recompute headTopY using current head position
     const headTopOffset = 0.55;
     tower.headTopY = tower.mesh.position.y + head.position.y + headTopOffset;
 
@@ -243,4 +239,4 @@ export default {
   onDestroy(tower) {
     this.removeLaser(tower);
   },
-};
+};