Ticket #3699: 3699_tooltip.2.patch

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

@@ -186 +186 @@
         {
             "Name": "skirmish",
             "Title": translateWithContext("map", "Skirmish"),
+            "Description": translate("A map with predefined landscape."),
             "Default": true
         },
         {
             "Name": "random",
-            "Title": translateWithContext("map", "Random")
+            "Title": translateWithContext("map", "Random"),
+            "Description": translate("A randomly generated map.")
         },
         {
             "Name": "scenario",
-            "Title": translateWithContext("map", "Scenario")
+            "Title": translateWithContext("map", "Scenario"),
+            "Description": translate("A map with predefined landscape, civilizations and players.")
         }
     ];
 }

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

@@ -322 +322 @@
         if (this.selected != -1)
             selectMapType(this.list_data[this.selected]);
     };
+    mapTypes.onHoverChange = function() {
+        mapTypes.tooltip = g_MapTypes.Description[this.hovered_item] ||
+            translate("Select a map type.");
+    }
     if (g_IsController)
         mapTypes.selected = g_MapTypes.Default;
 }
@@ -459 +463 @@
 
         updateGameAttributes();
     };
+    victoryConditions.onHoverChange = function() {
+        victoryConditions.tooltip = g_VictoryConditions.Description[this.hovered_item] ||
+            translate("Select victory condition.");
+    };
     victoryConditions.selected = g_VictoryConditions.Default;
 }
 

source/gui/CList.cpp

@@ -50 +50 @@
     // Each list item has both a name (in 'list') and an associated data string (in 'list_data')
     AddSetting(GUIST_CGUIList,              "list");
     AddSetting(GUIST_CGUIList,              "list_data"); // TODO: this should be a list of raw strings, not of CGUIStrings
+    AddSetting(GUIST_int,                   "hovered_item");
 
-    GUI<bool>::SetSetting(this, "scrollbar", false);
+    GUI<bool>::SetSetting(this,             "scrollbar", false);
+    GUI<int>::SetSetting(this,              "selected", -1);
+    GUI<int>::SetSetting(this,              "hovered_item", -1);
 
-    // Nothing is selected as default.
-    GUI<int>::SetSetting(this, "selected", -1);
-
     // Add scroll-bar
     CGUIScrollBarVertical* bar = new CGUIScrollBarVertical();
     bar->SetRightAligned(true);
@@ -228 +228 @@
         break;
     }
 
+    case GUIM_MOUSE_LEAVE:
+    {
+        GUI<int>::SetSetting(this, "hovered_item", -1);
+        ScriptEvent("hoverchange");
+        break;
+    }
+
+    case GUIM_MOUSE_OVER:
+    {
+        bool scrollbar;
+        CGUIList* pList;
+        GUI<bool>::GetSetting(this, "scrollbar", scrollbar);
+        GUI<CGUIList>::GetSettingPointer(this, "list", pList);
+        float scroll = 0.f;
+        if (scrollbar)
+            scroll = GetScrollBar(0).GetPos();
+
+        CRect rect = GetListRect();
+        CPos mouse = GetMousePos();
+        mouse.y += scroll;
+        int set = -1;
+        for (int i = 0; i < (int)pList->m_Items.size(); ++i)
+        {
+            if (mouse.y >= rect.top + m_ItemsYPositions[i] &&
+                mouse.y < rect.top + m_ItemsYPositions[i + 1] &&
+                // mouse is not over scroll-bar
+                (!scrollbar || !GetScrollBar(0).IsVisible() ||
+                mouse.x < GetScrollBar(0).GetOuterRect().left ||
+                mouse.x > GetScrollBar(0).GetOuterRect().right))
+            {
+                set = i;
+            }
+        }
+
+        GUI<int>::SetSetting(this, "hovered_item", set);
+        ScriptEvent("hoverchange");
+        break;
+    }
+
     case GUIM_LOAD:
     {
         CStr scrollbar_style;