Refactor Logger module usage and improve shift key input handling in GrappleGun

Improves GrappleGun shift input and refactors Logger path

- Enhances shift key detection for GrappleGun's precise rope control, using a more robust method for increased reliability.
- Updates GrappleGun scripts to reference a common, centralized path for the Logger module.
- Removes an obsolete Lua binding for the shift key, as direct shift state checking is now preferred.
- Includes minor code cleanup in the input mapping settings UI.

Author: OpenTools-run

Data/Base.rte/Devices/Tools/GrappleGun/Grapple.lua

@@ -7,7 +7,7 @@ local RopePhysics = require("Devices.Tools.GrappleGun.Scripts.RopePhysics")
 local RopeRenderer = require("Devices.Tools.GrappleGun.Scripts.RopeRenderer")
 local RopeInputController = require("Devices.Tools.GrappleGun.Scripts.RopeInputController")
 local RopeStateManager = require("Devices.Tools.GrappleGun.Scripts.RopeStateManager")
-local Logger = require("Devices.Tools.GrappleGun.Scripts.Logger")
+local Logger = require("Scripts.Logger")
 
 function Create(self)
 	Logger.info("Grapple Create() - Starting initialization")

Data/Base.rte/Devices/Tools/GrappleGun/Scripts/RopeInputController.lua

@@ -238,9 +238,9 @@ function RopeInputController.handleShiftMousewheelControls(grappleInstance, cont
 
 	Logger.debug("RopeInputController.handleShiftMousewheelControls() - Equipment and attachment checks passed")
 
-	-- Check for actual keyboard SHIFT key
-	local shiftHeld = controller:IsState(Controller.KEYBOARD_SHIFT)
-	Logger.debug("RopeInputController.handleShiftMousewheelControls() - Keyboard SHIFT held (KEYBOARD_SHIFT): %s", tostring(shiftHeld))
+	-- Check for actual SHIFT key using UInputMan
+	local shiftHeld = UInputMan:FlagShiftState()
+	Logger.debug("RopeInputController.handleShiftMousewheelControls() - SHIFT key held (UInputMan): %s", tostring(shiftHeld))
 
 	if not shiftHeld then
 		return false

Source/Lua/LuaBindingsInput.cpp

@@ -44,7 +44,6 @@ LuaBindingRegisterFunctionDefinitionForType(InputLuaBindings, InputElements) {
 	                            luabind::value("INPUT_JUMP", InputElements::INPUT_JUMP),
 	                            luabind::value("INPUT_CROUCH", InputElements::INPUT_PRONE), // awful, but script compat
 	                            luabind::value("INPUT_PRONE", InputElements::INPUT_PRONE),
-	                            luabind::value("INPUT_SHIFT", InputElements::INPUT_SHIFT),
 	                            luabind::value("INPUT_WALKCROUCH", InputElements::INPUT_CROUCH),
 	                            luabind::value("INPUT_NEXT", InputElements::INPUT_NEXT),
 	                            luabind::value("INPUT_PREV", InputElements::INPUT_PREV),

Source/Menus/SettingsInputMappingGUI.cpp

@@ -13,7 +13,7 @@ using namespace RTE;
 std::array<InputElements, 7> SettingsInputMappingGUI::m_InputElementsUsedByMouse = {InputElements::INPUT_FIRE, InputElements::INPUT_PIEMENU_ANALOG, InputElements::INPUT_AIM, InputElements::INPUT_AIM_UP, InputElements::INPUT_AIM_DOWN, InputElements::INPUT_AIM_LEFT, InputElements::INPUT_AIM_RIGHT};
 
 SettingsInputMappingGUI::SettingsInputMappingGUI(GUIControlManager* parentControlManager) :
-    m_GUIControlManager(parentControlManager) {
+	m_GUIControlManager(parentControlManager) {
 	m_InputMappingSettingsBox = dynamic_cast<GUICollectionBox*>(m_GUIControlManager->GetControl("CollectionBoxPlayerInputMapping"));
 	m_InputMappingSettingsBox->SetVisible(false);
 
@@ -40,25 +40,22 @@ SettingsInputMappingGUI::SettingsInputMappingGUI(GUIControlManager* parentContro
 		if (m_InputMapButton[i]) {
 			// Optionally, initialize button text or state here if needed
 		}
-		m_InputMapLabel[i]->SetText(c_InputElementNames[i]);
 	}
 
-	m_InputMapButton[i] = dynamic_cast<GUIButton*>(m_GUIControlManager->GetControl("ButtonInputKey" + std::to_string(i + 1)));
-}
-m_InputMappingCaptureBox = dynamic_cast<GUICollectionBox*>(m_GUIControlManager->GetControl("CollectionBoxInputCapture"));
-m_InputMappingCaptureBox->SetVisible(false);
+	m_InputMappingCaptureBox = dynamic_cast<GUICollectionBox*>(m_GUIControlManager->GetControl("CollectionBoxInputCapture"));
+	m_InputMappingCaptureBox->SetVisible(false);
 
-GUICollectionBox* settingsRootBox = dynamic_cast<GUICollectionBox*>(m_GUIControlManager->GetControl("CollectionBoxSettingsBase"));
-m_InputMappingCaptureBox->SetPositionAbs(settingsRootBox->GetXPos() + ((settingsRootBox->GetWidth() - m_InputMappingCaptureBox->GetWidth()) / 2), settingsRootBox->GetYPos() + ((settingsRootBox->GetHeight() - m_InputMappingCaptureBox->GetHeight()) / 2));
+	GUICollectionBox* settingsRootBox = dynamic_cast<GUICollectionBox*>(m_GUIControlManager->GetControl("CollectionBoxSettingsBase"));
+	m_InputMappingCaptureBox->SetPositionAbs(settingsRootBox->GetXPos() + ((settingsRootBox->GetWidth() - m_InputMappingCaptureBox->GetWidth()) / 2), settingsRootBox->GetYPos() + ((settingsRootBox->GetHeight() - m_InputMappingCaptureBox->GetHeight()) / 2));
 
-m_InputElementCapturingInputNameLabel = dynamic_cast<GUIButton*>(m_GUIControlManager->GetControl("ButtonLabelInputMappingName"));
+	m_InputElementCapturingInputNameLabel = dynamic_cast<GUIButton*>(m_GUIControlManager->GetControl("ButtonLabelInputMappingName"));
 
-m_InputConfigWizardMenu = std::make_unique<SettingsInputMappingWizardGUI>(parentControlManager);
+	m_InputConfigWizardMenu = std::make_unique<SettingsInputMappingWizardGUI>(parentControlManager);
 
-m_ConfiguringPlayer = Players::NoPlayer;
-m_ConfiguringPlayerInputScheme = nullptr;
-m_ConfiguringManually = false;
-m_InputElementCapturingInput = InputElements::INPUT_COUNT;
+	m_ConfiguringPlayer = Players::NoPlayer;
+	m_ConfiguringPlayerInputScheme = nullptr;
+	m_ConfiguringManually = false;
+	m_InputElementCapturingInput = InputElements::INPUT_COUNT;
 }
 
 bool SettingsInputMappingGUI::IsEnabled() const {