Patch Issue #81 (#82)

Fix: Anonymous Login button does not turn green after successful anonymous login

Making the button green which is currently logged in

Co-authored-by: Johannes Ebner <Structed@users.noreply.github.com>

Author: zymsbgt

Scenes/Login.gd

@@ -1,9 +1,28 @@
 extends MarginContainer
 
+const color_green = Color(0, 1, 0, 0.5)
+const color_white = Color(1, 1, 1, 1)
+const color_red = Color(1, 0, 0, 0.5)
 
 func _ready():
 	var _error = PlayFabManager.client.connect("api_error", self, "_on_api_error")
 	_error = PlayFabManager.client.connect("logged_in", self, "_on_logged_in")
+	update_login_button_states()
+
+
+func update_login_button_states():
+	# Reset button colours
+	$Login/Login.self_modulate = color_white
+	$Login/AnonLogin.self_modulate = color_white
+
+	if PlayFabManager.client_config.is_logged_in():
+		match PlayFabManager.client_config.login_type:
+			PlayFabClientConfig.LoginType.LOGIN_CUSTOM_ID:
+				$Login/AnonLogin.self_modulate = color_green
+			PlayFabClientConfig.LoginType.LOGIN_EMAIL:
+				$Login/Login.self_modulate = color_green
+			_:
+				pass # No specific action needed
 
 
 func _on_Login_pressed():
@@ -43,7 +62,6 @@ func _hide_progess():
 
 
 func _on_logged_in(login_result: LoginResult):
-	$Login/Login.self_modulate = Color(0, 1, 0, 0.5)
 	$Login/Output.hide()
 	_hide_progess()
 
@@ -63,7 +81,7 @@ func _on_api_error(api_error_wrapper: ApiErrorWrapper):
 			for element in error_details[key]:
 				text += "%s\n" % element
 
-	$Login/Login.self_modulate = Color(1, 0, 0, 0.5)
+	$Login/Login.self_modulate = color_red
 	$Login/Output.show()
 	$Login/Output.bbcode_text = text
 
@@ -74,5 +92,5 @@ func _on_Back_pressed():
 
 func _on_LoggedInBackButton_pressed():
 	$LoggedIn.hide()
+	update_login_button_states()
 	$Login.show()
-	pass