Ticket #4279: vision_range_tooltip.diff

binaries/data/mods/public/simulation/components/GuiInterface.js

@@ -248 +248 @@
         "trader": null,
         "unitAI": null,
         "visibility": null,
     };
 
+    let cmpVision = Engine.QueryInterface(ent, IID_Vision);
+    if (cmpVision)
+        ret.vision = {
+            "range": cmpVision.GetRange()
+        };
+
     let cmpMirage = Engine.QueryInterface(ent, IID_Mirage);
     if (cmpMirage)
         ret.mirage = true;
 
     let cmpIdentity = Engine.QueryInterface(ent, IID_Identity);

binaries/data/mods/public/gui/session/selection_details.js

@@ -281 +281 @@
 
     // TODO: we should require all entities to have icons
     Engine.GetGUIObjectByName("icon").sprite = template.icon ? ("stretched:session/portraits/" + template.icon) : "BackgroundBlack";
 
     Engine.GetGUIObjectByName("attackAndArmorStats").tooltip = [
+        getVisionTooltip,
         getAttackTooltip,
         getSplashDamageTooltip,
         getHealerTooltip,
         getArmorTooltip,
         getRepairRateTooltip,

binaries/data/mods/public/gui/common/tooltips.js

@@ -130 +130 @@
             "damage": dmg[dmgType].toFixed(1),
             "damageType": unitFont(g_DamageTypes[dmgType])
         })).join(commaFont(translate(", ")));
 }
 
+function getVisionTooltip(template)
+{
+    if (!template.vision)
+        return "";
+
+    let range = template.vision.range;
+
+    return sprintf(translate("%(label)s %(details)s"), {
+        "label": translate("Vision Range"),
+        "details": sprintf(translatePlural("%(range)s %(meters)s", "%(range)s %(meters)s", range), {
+            "range": range,
+            "meters": unitFont(translatePlural("meter", "meters", range))
+        })
+    });
+}
+
 // TODO: should also show minRange
 function getAttackTooltip(template)
 {
     if (!template.attack)
         return "";