Ticket #3000: heropanel.patch

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

@@ -8 +8 @@
 const g_PopulationAlertColor = "orange";
 
 /**
+ * Maximum number of heroes to display (must stay in sync with hero_icon.xml).
+ */
+const g_HeroPanelCount = 10;
+
+/**
  * A random file will be played. TODO: more variety
  */
 const g_Ambient = [ "audio/ambient/dayscape/day_temperate_gen_03.ogg" ];
@@ -145 +150 @@
 /**
  * Blink the hero selection if that entity has lost health since the last turn.
  */
-var g_PreviousHeroHitPoints;
+var g_PreviousHeroes = [];
 
 /**
  * Unit classes to be checked for the idle-worker-hotkey.
@@ -712 +717 @@
     if (g_ShowGuarding || g_ShowGuarded)
         updateAdditionalHighlight();
 
-    updateHero();
+    updateHeroes();
     updateGroups();
     updateDebug();
     updatePlayerDisplay();
@@ -788 +793 @@
 }
 
 
-function updateHero()
+function updateHeroes()
 {
     let unitHeroPanel = Engine.GetGUIObjectByName("unitHeroPanel");
-    let heroButton = Engine.GetGUIObjectByName("unitHeroButton");
-
     let playerState = GetSimState().players[g_ViewedPlayer];
     if (!playerState || playerState.heroes.length <= 0)
     {
-        g_PreviousHeroHitPoints = undefined;
+        g_PreviousHeroes = [];
         unitHeroPanel.hidden = true;
         return;
     }
 
-    let heroImage = Engine.GetGUIObjectByName("unitHeroImage");
-    let heroState = GetExtendedEntityState(playerState.heroes[0]);
-    let template = GetTemplateData(heroState.template);
-    heroImage.sprite = "stretched:session/portraits/" + template.icon;
-    let hero = playerState.heroes[0];
+    unitHeroPanel.hidden = false;
 
-    heroButton.onpress = function()
+    let usedSlots = [];
+    for(let i = 0; i < g_HeroPanelCount; ++i)
+        usedSlots.push(false);
+        
+    let mapped = playerState.heroes.map(ent =>
+        {
+            // Map to current button, if available.
+            if (g_PreviousHeroes[ent])
+                return { 
+                    "ent": ent, 
+                    "index": g_PreviousHeroes[ent].displayIndex, 
+                    "previous": g_PreviousHeroes[ent] };
+            // Using the panel count places new heroes at the end of the list.
+            return { 
+                "ent": ent, 
+                "index": g_HeroPanelCount };
+        }).sort((l, r) => l.index == r.index ? l.ent - r.ent : l.index - r.index);
+    
+    let heroes = [];
+    
+    for (let displayIndex = 0; displayIndex < Math.min(g_HeroPanelCount, mapped.length); ++displayIndex)
     {
-        if (!Engine.HotkeyIsPressed("selection.add"))
-            g_Selection.reset();
-        g_Selection.addList([hero]);
-    };
-    heroButton.ondoublepress = function() { selectAndMoveTo(getEntityOrHolder(hero)); };
-    unitHeroPanel.hidden = false;
+        let hero = mapped[displayIndex];
+        // Find the first unused one if new, otherwise reuse previous.
+        let buttonSlot = hero.previous === undefined ? usedSlots.findIndex(slot => !slot) : hero.previous.button;
+        usedSlots[buttonSlot] = true;
+        
+        let heroButton = Engine.GetGUIObjectByName("unitHeroButton[" + buttonSlot + "]");
+        let heroState = GetExtendedEntityState(hero.ent);
+        let template = GetTemplateData(heroState.template);
+        
+        if (hero.previous === undefined)
+        {
+            heroButton.onpress = function()
+            {
+                if (!Engine.HotkeyIsPressed("selection.add"))
+                    g_Selection.reset();
+                g_Selection.addList([hero.ent]);
+            };
+            heroButton.ondoublepress = function() { selectAndMoveTo(getEntityOrHolder(hero.ent)); };
+            heroButton.hidden = false;
+            // Ensure no pre-existing animations.
+            stopColorFade("heroHitOverlay[" + buttonSlot + "]");
+            let heroImage = Engine.GetGUIObjectByName("unitHeroImage[" + buttonSlot + "]");
+            heroImage.sprite = "stretched:session/portraits/" + template.icon;
+        }
+
+        heroButton.tooltip = createHeroTooltip(heroState, template);
+        updateGUIStatusBar("heroHealthBar[" + buttonSlot + "]", heroState.hitpoints, heroState.maxHitpoints);
+        setPanelObjectPosition(heroButton, displayIndex, g_HeroPanelCount);
+
+        // if the health of the hero changed since the last update, trigger the animation
+        if (hero.previous !== undefined && hero.previous.player == g_ViewedPlayer && hero.previous.hitpoints > heroState.hitpoints)
+            startColorFade("heroHitOverlay[" + buttonSlot + "]", 100, 0, colorFade_attackUnit, true, smoothColorFadeRestart_attackUnit);
+
+        heroes[hero.ent] = {
+            "displayIndex": displayIndex,
+            "button": buttonSlot,
+            "hitpoints": heroState.hitpoints,
+            "player": g_ViewedPlayer };
+    }
+
+    // In case heroes were removed and their slot/button was freed, hide it.
+    for (let hero of g_PreviousHeroes.filter(prev => !usedSlots[prev.button]))
+    {
+        Engine.GetGUIObjectByName("unitHeroButton[" + hero.button + "]").hidden = true;
+    }
 
+    g_PreviousHeroes = heroes;
+}
+
+function createHeroTooltip(heroState, template)
+{
     // Setup tooltip
     let tooltip = "[font=\"sans-bold-16\"]" + template.name.specific + "[/font]";
-    let healthLabel = "[font=\"sans-bold-13\"]" + translate("Health:") + "[/font]";
-    tooltip += "\n" + sprintf(translate("%(label)s %(current)s / %(max)s"), { 
-        "label": healthLabel,
+    tooltip += "\n" + sprintf(translate("%(label)s %(current)s / %(max)s"), {
+        "label": "[font=\"sans-bold-13\"]" + translate("Health:") + "[/font]",
         "current": Math.ceil(heroState.hitpoints),
         "max": Math.ceil(heroState.maxHitpoints)
     });
@@ -830 +892 @@
     tooltip += "\n" + getArmorTooltip(heroState.armour);
     if (template.tooltip)
         tooltip += "\n" + template.tooltip;
-
-    heroButton.tooltip = tooltip;
-
-    // update heros health bar
-    updateGUIStatusBar("heroHealthBar", heroState.hitpoints, heroState.maxHitpoints);
-
-    let heroHP = {
-        "hitpoints": heroState.hitpoints,
-        "player": g_ViewedPlayer
-    };
-
-    if (!g_PreviousHeroHitPoints)
-        g_PreviousHeroHitPoints = heroHP;
-
-    // if the health of the hero changed since the last update, trigger the animation
-    if (g_PreviousHeroHitPoints.player == heroHP.player && g_PreviousHeroHitPoints.hitpoints > heroHP.hitpoints)
-        startColorFade("heroHitOverlay", 100, 0, colorFade_attackUnit, true, smoothColorFadeRestart_attackUnit);
-
-    g_PreviousHeroHitPoints = heroHP;
+    return tooltip;
 }
 
 function updateGroups()

binaries/data/mods/public/gui/session/session_objects/hero_icon.xml

@@ -4 +4 @@
     size="0 36 50 93"
     hidden="true"
 >
-    <object name="unitHeroButton" size="0 0 50 50" type="button" style="iconButton" 
-        tooltip_style="sessionToolTip">
-        <object name="unitHeroImage" size="5 5 100%-5 100%-5" type="image" ghost="true"/>
-        <object name="heroHitOverlay" hidden="true" type="image" ghost="true" size="5 5 100%-5 100%-5"/> 
-    </object>
-    <!-- Hero Health bar -->
-    <object size="3 100%-7 100%-3 100%-2" name="heroHealthSection" ghost="true">
-        <object size="0 0 100% 5" name="heroHealth" type="image" ghost="true">
-            <object type="image" sprite="barBorder" ghost="true" size="-1 -1 100%+1 100%+1"/>
-            <object type="image" sprite="healthBackground" ghost="true"/> 
-            <object type="image" sprite="healthForeground" ghost="true" name="heroHealthBar"/> 
-            <object type="image" sprite="statsBarShaderHorizontal" ghost="true"/>
+    <repeat count="10" var="n">
+        <object name="unitHeroButton[n]" size="0 0 50 50" type="button" hidden="true" style="iconButton" tooltip_style="sessionToolTip">
+            <object name="unitHeroImage[n]" size="5 5 100%-5 100%-5" type="image" ghost="true"/>
+            <object name="heroHitOverlay[n]" hidden="true" type="image" ghost="true" size="5 5 100%-5 100%-5"/> 
+            <!-- Hero Health bar -->
+            <object size="3 100%-7 100%-3 100%-2" name="heroHealthSection[n]" ghost="true">
+                <object size="0 0 100% 5" name="heroHealth[n]" type="image" ghost="true">
+                    <object type="image" sprite="barBorder" ghost="true" size="-1 -1 100%+1 100%+1"/>
+                    <object type="image" sprite="healthBackground" ghost="true"/> 
+                    <object type="image" sprite="healthForeground" ghost="true" name="heroHealthBar[n]"/> 
+                    <object type="image" sprite="statsBarShaderHorizontal" ghost="true"/>
+                </object>
+            </object>
         </object>
-    </object>
+    </repeat>
 </object>