Skip to content

Commit e2c48f9

Browse files
simplified cancellation callbacks
1 parent 8791b95 commit e2c48f9

File tree

3 files changed

+26
-10
lines changed

3 files changed

+26
-10
lines changed

Assets/BossRoom/Prefabs/UI/IPPopup.prefab

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2901,6 +2901,19 @@ MonoBehaviour:
29012901
m_StringArgument:
29022902
m_BoolArgument: 0
29032903
m_CallState: 2
2904+
- m_Target: {fileID: 5705220060289623588}
2905+
m_TargetAssemblyTypeName: Unity.Multiplayer.Samples.BossRoom.Visual.IPUIMediator,
2906+
Unity.Multiplayer.Samples.BossRoom.Client
2907+
m_MethodName: CancelConnectingWindow
2908+
m_Mode: 1
2909+
m_Arguments:
2910+
m_ObjectArgument: {fileID: 0}
2911+
m_ObjectArgumentAssemblyTypeName: UnityEngine.Object, UnityEngine
2912+
m_IntArgument: 0
2913+
m_FloatArgument: 0
2914+
m_StringArgument:
2915+
m_BoolArgument: 0
2916+
m_CallState: 2
29042917
--- !u!1 &5588059588747898398
29052918
GameObject:
29062919
m_ObjectHideFlags: 0

Assets/BossRoom/Scripts/Client/UI/IPConnectionWindow.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ void OnTimeElapsed()
4646
PopupManager.ShowPopupPanel("Connection Failed",
4747
"Failed to connect to server and/or invalid network endpoint");
4848
Hide();
49-
m_IPUIMediator.ConnectingFinished();
49+
m_IPUIMediator.DisableSignInSpinner();
5050
}
5151

5252
switch (chosenTransport)
@@ -102,8 +102,7 @@ IEnumerator DisplayUTPReconnectAttempts(int maxReconnectAttempts, int connectTim
102102
// invoked by UI cancel button
103103
public void OnCancelJoinButtonPressed()
104104
{
105-
m_IPUIMediator.RequestShutdown();
106-
m_IPUIMediator.ConnectingFinished();
105+
m_IPUIMediator.JoiningWindowCancelled();
107106
End();
108107
}
109108
}

Assets/BossRoom/Scripts/Client/UI/IPUIMediator.cs

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using System;
2-
using System.Runtime.CompilerServices;
32
using System.Text.RegularExpressions;
43
using TMPro;
54
using Unity.Multiplayer.Samples.BossRoom.Client;
@@ -105,17 +104,18 @@ public void JoinWithIP(string ip, string port)
105104
m_IPConnectionWindow.ShowConnectingWindow();
106105
}
107106

108-
void CancelConnectingWindow()
107+
public void JoiningWindowCancelled()
109108
{
110-
m_IPConnectionWindow.Cancel();
109+
DisableSignInSpinner();
110+
RequestShutdown();
111111
}
112112

113-
public void ConnectingFinished()
113+
public void DisableSignInSpinner()
114114
{
115115
m_SignInSpinner.SetActive(false);
116116
}
117117

118-
public void RequestShutdown()
118+
void RequestShutdown()
119119
{
120120
if (m_GameNetPortal && m_GameNetPortal.NetManager && m_GameNetPortal.NetManager.IsListening)
121121
{
@@ -154,17 +154,21 @@ public void Show()
154154
m_CanvasGroup.interactable = true;
155155
m_CanvasGroup.blocksRaycasts = true;
156156

157-
m_SignInSpinner.SetActive(false);
157+
DisableSignInSpinner();
158158
}
159159

160160
public void Hide()
161161
{
162162
m_CanvasGroup.alpha = 0f;
163163
m_CanvasGroup.interactable = false;
164164
m_CanvasGroup.blocksRaycasts = false;
165+
}
165166

166-
CancelConnectingWindow();
167+
// To be called from the Cancel (X) UI button
168+
public void CancelConnectingWindow()
169+
{
167170
RequestShutdown();
171+
m_IPConnectionWindow.Cancel();
168172
}
169173

170174
/// <summary>

0 commit comments

Comments
 (0)