From fc10d600a502ac45abd85cab40c17187cb8e4e4a Mon Sep 17 00:00:00 2001
From: Emmanouil Papadeas <35376950+OverloadedOrama@users.noreply.github.com>
Date: Fri, 14 Jun 2024 16:33:52 +0300
Subject: [PATCH] Remove Crop tool's apply button shortcut

The issue is that by default it's mapped to Enter, which is also used to confirm editing of a slider. If we want to keep the shortcut, we need to check if something else does not have focus first.
---
 src/Tools/UtilityTools/CropTool.tscn | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/Tools/UtilityTools/CropTool.tscn b/src/Tools/UtilityTools/CropTool.tscn
index 4d45d3363..9e7642f31 100644
--- a/src/Tools/UtilityTools/CropTool.tscn
+++ b/src/Tools/UtilityTools/CropTool.tscn
@@ -1,4 +1,4 @@
-[gd_scene load_steps=8 format=3 uid="uid://c35n21ii7onhe"]
+[gd_scene load_steps=6 format=3 uid="uid://c35n21ii7onhe"]
 
 [ext_resource type="PackedScene" uid="uid://ctfgfelg0sho8" path="res://src/Tools/BaseTool.tscn" id="1"]
 [ext_resource type="PackedScene" uid="uid://yjhp0ssng2mp" path="res://src/UI/Nodes/ValueSlider.tscn" id="2"]
@@ -6,12 +6,6 @@
 [ext_resource type="Texture2D" uid="uid://bgrq56ndc4ydj" path="res://assets/graphics/misc/unlocked_size.png" id="4"]
 [ext_resource type="PackedScene" path="res://src/UI/Nodes/ValueSliderV2.tscn" id="5"]
 
-[sub_resource type="InputEventAction" id="InputEventAction_inhdu"]
-action = &"transformation_confirm"
-
-[sub_resource type="Shortcut" id="Shortcut_5ydch"]
-events = [SubResource("InputEventAction_inhdu")]
-
 [node name="ToolOptions" instance=ExtResource("1")]
 script = ExtResource("3")
 
@@ -145,7 +139,6 @@ prefix_y = "Height:"
 [node name="Apply" type="Button" parent="." index="7"]
 layout_mode = 2
 mouse_default_cursor_shape = 2
-shortcut = SubResource("Shortcut_5ydch")
 text = "Apply"
 
 [node name="DimensionsLabel" type="Label" parent="." index="8"]