Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pressing TAB in Inspector fields moves the focus upwards #57854

Closed
timothyqiu opened this issue Feb 9, 2022 · 1 comment
Closed

Pressing TAB in Inspector fields moves the focus upwards #57854

timothyqiu opened this issue Feb 9, 2022 · 1 comment

Comments

@timothyqiu
Copy link
Member

Godot version

3.x (4ed2c8e)

System information

Arch Linux, GLES 3, Mesa Intel(R) UHD Graphics (CML GT2)

Issue description

Pressing TAB in the video:

Peek.2022-02-09.20-53.mp4

Bisecting shows it's introduced in #54174. Reverting the logic changes in core/input_map.cpp fixes the issue:

diff --git i/core/input_map.cpp w/core/input_map.cpp
index 6dd8523377..30f39b57ac 100644
--- i/core/input_map.cpp
+++ w/core/input_map.cpp
@@ -136,7 +136,9 @@ List<Ref<InputEvent>>::Element *InputMap::_find_event(Action &p_action, const Re
 
 		int device = e->get_device();
 		if (device == ALL_DEVICES || device == p_event->get_device()) {
-			if (e->action_match(p_event, p_exact_match, p_pressed, p_strength, p_raw_strength, p_action.deadzone)) {
+			if (p_exact_match && e->shortcut_match(p_event)) {
+				return E;
+			} else if (!p_exact_match && e->action_match(p_event, true, p_pressed, p_strength, p_raw_strength, p_action.deadzone)) {
 				return E;
 			}
 		}

Not sure if the changes are intended. CC @nathanfranke

Inspector tabbing on master has been broken in another way for quite sometime (#53832), so not reproducible there. This report is only for 3.x.

Steps to reproduce

Make the Inspector inspecting something, click inside a field, and press TAB.

Minimal reproduction project

No response

@timothyqiu timothyqiu changed the title [3.x] Pressing TAB in Inspector fields moves the focus upwards Pressing TAB in Inspector fields moves the focus upwards Feb 9, 2022
@Calinou Calinou added this to the 4.0 milestone Feb 9, 2022
@akien-mga akien-mga modified the milestones: 4.0, 3.5 Feb 9, 2022
@akien-mga
Copy link
Member

Fixed by #57864.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants