Ticket #4112: hidereadybutton3.patch

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

@@ -522 +522 @@
         g_GameAttributes.settings.LastManStanding = false;
         updateGameAttributes();
     };
 }
 
+function hideStartGameButton(hidden)
+{
+    Engine.GetGUIObjectByName("startGame").hidden = hidden;
+
+    // Move the cheatWarningText and back button to the right when hidden.
+    Engine.GetGUIObjectByName("cancelGame").size = hidden ? "100%-164 100%-52 100%-24 100%-24" : "100%-308 100%-52 100%-168 100%-24";
+    Engine.GetGUIObjectByName("cheatWarningText").size = hidden ? "0 100%-52 100%-176 100%-24" : "0 100%-52 100%-320 100%-24";
+
+    // Increase the size of onscreenToolTip when hidden.
+    Engine.GetGUIObjectByName("onscreenToolTip").size = hidden? "20 100%-56 100%-168 100%-24" : "20 100%-56 100%-312 100%-24";
+}
+
 /**
  * If we're a network client, hide the controls and show the text instead.
  */
 function hideControls()
 {
@@ -538 +550 @@
         Engine.GetGUIObjectByName("playerAssignment["+i+"]").hidden = true;
         Engine.GetGUIObjectByName("playerCiv["+i+"]").hidden = true;
         Engine.GetGUIObjectByName("playerTeam["+i+"]").hidden = true;
     }
 
+    // The start game button should be hidden until the player assignments are received
+    // and it is known whether the local player is an observer.
+    hideStartGameButton(true);
     Engine.GetGUIObjectByName("startGame").enabled = true;
 }
 
 /**
  * Hides the GUI controls for clients and shows the read-only label instead.
@@ -728 +743 @@
         if (!message.newAssignments[guid])
             onClientLeave(guid);
 
     g_PlayerAssignments = message.newAssignments;
 
+    hideStartGameButton(!g_IsController && g_PlayerAssignments[Engine.GetPlayerGUID()].player == -1);
+
     updatePlayerList();
     updateReadyUI();
     sendRegisterGameStanza();
 }