DamagedByTerrain #12666Here is a WIP branch to solve these issues but I can't promise it is the right way to do this.
A simpler fix is to disable it using a condition, which is what I did for several other traits. We want to avoid having to add a code dependency from every trait onto the tunnel logic.
Issue 2 is complex, units in tunnels were always shown by health bar in TS and detected enemy subterranean units also revealed themselves via them.
I was planning on repurposing our RA GPS icons (but not the code) for friendly units in tunnels and enemy units detected with sensors. IMO that will be much more polished than showing just the health bars.
@pchote That would indeed be much easier.
I'll give the DamagedByTerrains the !in-tunnel condition.
Point 3 was solved by #13149, and the second point is covered by #12913.
Most helpful comment
I was planning on repurposing our RA GPS icons (but not the code) for friendly units in tunnels and enemy units detected with sensors. IMO that will be much more polished than showing just the health bars.