From 78afbafe2a2accf67d65155fc03fd10c13df3973 Mon Sep 17 00:00:00 2001 From: SkyratBot <59378654+SkyratBot@users.noreply.github.com> Date: Mon, 24 Jul 2023 07:53:05 +0200 Subject: [PATCH] [MIRROR] ID card screentips now show you how to assign an account to an ID. [MDB IGNORE] (#22657) * ID card screentips now show you how to assign an account to an ID. (#76998) ## About The Pull Request Based on discussion regarding player bank accounts and prisoners, I was sure that Id cards had screentips, and it turns out I was right. However, the screentips are inaccurate. Alt Click serves a dual purpose. In an ID card with only a default storage account, alt-click will ask you to assign a new account to it. After, you can remove credits from it after it's been assigned. Apparently players have been confused about assigning their account to an ID card, due to not having a proper screentip telling them how. This updates the screentip to show both functions. ## Why It's Good For The Game ![image](https://github.com/tgstation/tgstation/assets/41715314/346269d2-0224-4bf6-8d6a-fef7e52e1926) Improves visual clarity and better teaches the mechanics related to economy. ## Changelog :cl: qol: ID cards tooltips now show how to assign a new account. /:cl: --------- Co-authored-by: san7890 * ID card screentips now show you how to assign an account to an ID. --------- Co-authored-by: ArcaneMusic <41715314+ArcaneMusic@users.noreply.github.com> Co-authored-by: san7890 --- code/game/objects/items/cards_ids.dm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/code/game/objects/items/cards_ids.dm b/code/game/objects/items/cards_ids.dm index 7146d50c265..b94b31f3654 100644 --- a/code/game/objects/items/cards_ids.dm +++ b/code/game/objects/items/cards_ids.dm @@ -436,7 +436,10 @@ context[SCREENTIP_CONTEXT_LMB] = "Show ID" context[SCREENTIP_CONTEXT_RMB] = "Project pay stand" - context[SCREENTIP_CONTEXT_ALT_LMB] = "Withdraw credits" + if(isnull(registered_account) || registered_account.replaceable) //Same check we use when we check if we can assign an account + context[SCREENTIP_CONTEXT_ALT_LMB] = "Assign account" + else + context[SCREENTIP_CONTEXT_ALT_LMB] = "Withdraw credits" return CONTEXTUAL_SCREENTIP_SET /obj/item/card/id/proc/try_project_paystand(mob/user, turf/target)