From bb7ba24577e916013f553a2eca6d860d72b24863 Mon Sep 17 00:00:00 2001
From: _0Steven <42909981+00-Steven@users.noreply.github.com>
Date: Mon, 3 Mar 2025 13:48:46 +0100
Subject: [PATCH] Add screentips to big manipulators, move wire interactions to
 is_wire_tool(...) usage (#89755)

## About The Pull Request

While working on another pr I noticed big manipulators didn't have
screentips yet, so here we add those.
In the process of doing _that_, I noticed we do some awkward usage of
`wirecutter/multitool_act(...)` for interacting with wires, which
doesn't actually cover the assembly use case of interacting with wires.
So to solve this, we move it to checking for `is_wire_tool(...)` on
`item_interaction(...)` instead, which actually runs for all of our
wire-interacting items.
## Why It's Good For The Game

screentips make everything less obtuse
being able to actually open the menu with all the items we're supposed
to be able to open the menu with is good, and the code is slightly less
cluttered from having the wire tool logic in one spot than y'know
several procs
## Changelog
:cl:
qol: Added screentips to the big manipulator.
fix: You can actually open the big manipulator's wires UI while holding
an assembly.
/:cl:
---
 code/game/machinery/big_manipulator.dm | 43 +++++++++++++++++---------
 1 file changed, 28 insertions(+), 15 deletions(-)

diff --git a/code/game/machinery/big_manipulator.dm b/code/game/machinery/big_manipulator.dm
index 96f7418c2db7e..f4d04656b3276 100644
--- a/code/game/machinery/big_manipulator.dm
+++ b/code/game/machinery/big_manipulator.dm
@@ -80,6 +80,8 @@
 		press_on(pressed_by = null)
 	set_wires(new /datum/wires/big_manipulator(src))
 
+	register_context()
+
 /// Init priority settings list for all modes.
 /obj/machinery/big_manipulator/proc/set_up_priority_settings()
 	for(var/datum/manipulator_priority/priority_for_drop as anything in subtypesof(/datum/manipulator_priority/for_drop))
@@ -95,6 +97,27 @@
 	if(!isnull(monkey_resolve))
 		. += "You can see [monkey_resolve]: [src] manager."
 
+/obj/machinery/big_manipulator/add_context(atom/source, list/context, obj/item/held_item, mob/user)
+	. = ..()
+
+	if(isnull(held_item))
+		context[SCREENTIP_CONTEXT_LMB] = panel_open ? "Interact with wires" : "Open UI"
+		return CONTEXTUAL_SCREENTIP_SET
+
+	if(held_item.tool_behaviour == TOOL_WRENCH)
+		context[SCREENTIP_CONTEXT_LMB] = "[anchored ? "Una" : "A"]nchor"
+		context[SCREENTIP_CONTEXT_RMB] = "Rotate clockwise"
+		return CONTEXTUAL_SCREENTIP_SET
+	if(held_item.tool_behaviour == TOOL_SCREWDRIVER)
+		context[SCREENTIP_CONTEXT_LMB] = "[panel_open ? "Close" : "Open"] panel"
+		return CONTEXTUAL_SCREENTIP_SET
+	if(held_item.tool_behaviour == TOOL_CROWBAR && panel_open)
+		context[SCREENTIP_CONTEXT_LMB] = "Deconstruct"
+		return CONTEXTUAL_SCREENTIP_SET
+	if(is_wire_tool(held_item) && panel_open)
+		context[SCREENTIP_CONTEXT_LMB] = "Interact with wires"
+		return CONTEXTUAL_SCREENTIP_SET
+
 /obj/machinery/big_manipulator/Destroy(force)
 	. = ..()
 	qdel(manipulator_hand)
@@ -176,24 +199,14 @@
 		return ITEM_INTERACT_SUCCESS
 	return ITEM_INTERACT_BLOCKING
 
-/obj/machinery/big_manipulator/multitool_act(mob/living/user, obj/item/multitool/tool)
-	if(!panel_open)
-		return ITEM_INTERACT_BLOCKING
+/obj/machinery/big_manipulator/item_interaction(mob/living/user, obj/item/tool, list/modifiers)
+	if(user.combat_mode)
+		return NONE
+	if(!panel_open || !is_wire_tool(tool))
+		return NONE
 	wires.interact(user)
 	return ITEM_INTERACT_SUCCESS
 
-/obj/machinery/big_manipulator/multitool_act_secondary(mob/living/user, obj/item/tool)
-	return multitool_act(user, tool)
-
-/obj/machinery/big_manipulator/wirecutter_act(mob/living/user, obj/item/tool)
-	if(!panel_open)
-		return ITEM_INTERACT_BLOCKING
-	wires.interact(user)
-	return ITEM_INTERACT_SUCCESS
-
-/obj/machinery/big_manipulator/wirecutter_act_secondary(mob/living/user, obj/item/tool)
-	return wirecutter_act(user, tool)
-
 /obj/machinery/big_manipulator/RefreshParts()
 	. = ..()