Ticket #3244: t3244_keep_player_and_gamelist_selection_a18.patch

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

@@ -1 +1 @@
 var g_ChatMessages = [];
 var g_Name = "unknown";
 var g_GameList = {};
+var g_selectedNick = "";
 var g_specialKey = Math.random();
 // This object looks like {"name":[numMessagesSinceReset, lastReset, timeBlocked]} when in use.
 var g_spamMonitor = {};
+var g_selectedGameIP = "";
 var g_timestamp = Engine.ConfigDB_GetValue("user", "lobby.chattimestamp") == "true";
 var g_mapSizes = {};
 const g_mapTypesText = [translateWithContext("map", "Skirmish"), translateWithContext("map", "Random"), translate("Scenario")];
 const g_mapTypes = ["skirmish", "random", "scenario"];
 var g_userRating = ""; // Rating of user, defaults to Unrated
@@ -153 +155 @@
  * @return Array containing the player, presence, nickname, and rating listings.
  */
 function updatePlayerList()
 {
     var playersBox = Engine.GetGUIObjectByName("playersBox");
+    
+    if (playersBox.selected > -1)
+        g_selectedNick = playersBox.list[playersBox.selected];
+        
     var playerList = [];
     var presenceList = [];
     var nickList = [];
     var ratingList = [];
     var cleanPlayerList = Engine.GetPlayerList();
@@ -188 +194 @@
     }
     playersBox.list_name = playerList;
     playersBox.list_status = presenceList;
     playersBox.list_rating = ratingList;
     playersBox.list = nickList;
-    if (playersBox.selected >= playersBox.list.length)
-        playersBox.selected = -1;
+    
+    // Select the previously selected player
+    playersBox.selected = nickList.indexOf(g_selectedNick);
+    
     return [playerList, presenceList, nickList, ratingList];
 }
 
 /**
  * Display the profile of the selected player.
@@ -355 +363 @@
  */
 function updateGameList()
 {
     var gamesBox = Engine.GetGUIObjectByName("gamesBox");
     var gameList = Engine.GetGameList();
+    if (gamesBox.selected > -1)
+        g_selectedGameIP = g_GameList[gamesBox.selected].ip;
+    
     // Store the game whole game list data so that we can access it later
     // to update the game info panel.
     g_GameList = gameList;
 
     // Sort the list of games to that games 'waiting' are displayed at the top, followed by 'init', followed by 'running'.
@@ -384 +395 @@
     var list_mapType = [];
     var list_nPlayers = [];
     var list = [];
     var list_data = [];
 
+    var selectedIndex = -1;
     var c = 0;
     for each (var g in gameList)
     {
         if(!filterGame(g))
         {
@@ -399 +411 @@
                 name = '[color="255 127 0"]' + g.name + '[/color]';
             else
                 name = '[color="255 0 0"]' + g.name + '[/color]';
             list_name.push(name);
             list_ip.push(g.ip);
+            if (g.ip == g_selectedGameIP)
+                selectedIndex = list_ip.length - 1;
             list_mapName.push(translate(g.niceMapName));
             list_mapSize.push(translatedMapSize(g.mapSize));
             let idx = g_mapTypes.indexOf(g.mapType);
             list_mapType.push(idx != -1 ? g_mapTypesText[idx] : "");
             list_nPlayers.push(g.nbp + "/" +g.tnbp);
@@ -418 +432 @@
     gamesBox.list_mapType = list_mapType;
     gamesBox.list_nPlayers = list_nPlayers;
     gamesBox.list = list;
     gamesBox.list_data = list_data;
 
-    if (gamesBox.selected >= gamesBox.list_name.length)
-        gamesBox.selected = -1;
-
+    // Select previously selected game
+    gamesBox.selected = selectedIndex;
+    
     // Update info box about the game currently selected
     updateGameSelection();
 }
 
 /**
@@ -626 +640 @@
             var playerList = playersBox.list_name;
             var presenceList = playersBox.list_status;
             var nickList = playersBox.list;
             var ratingList = playersBox.list_rating;
             var nickIndex = nickList.indexOf(nick);
+
+            if (playersBox.selected > -1)
+                g_selectedNick = nickList[playersBox.selected];
+
             switch(message.level)
             {
             case "join":
                 if (nick == g_Name)
                 {
@@ -655 +673 @@
                 addChatMessage({ "text": "/special " + sprintf(translate("%(nick)s has left."), { nick: nick }), "key": g_specialKey });
                 break;
             case "nick":
                 if (nickIndex == -1) // This shouldn't ever happen
                     break;
+                
                 if (!isValidNick(message.data))
                 {
                     addChatMessage({ "from": "system", "text": sprintf(translate("Invalid nickname: %(nick)s"), { nick: message.data })});
                     break;
                 }
@@ -687 +706 @@
             // Push new data to GUI
             playersBox.list_name = playerList;
             playersBox.list_status = presenceList;
             playersBox.list_rating = ratingList;
             playersBox.list = nickList;     
-            if (playersBox.selected >= playersBox.list.length)
-                playersBox.selected = -1;
+            // Select the previously selected player
+            playersBox.selected = nickList.indexOf(g_selectedNick);
             break;
         case "system":
             switch (message.level)
             {
             case "standard":