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

Fix Android input routing logic when using a hardware keyboard #80932

Merged

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Aug 23, 2023

When a hardware keyboard is connected, all key events come through so we can route them directly to the engine. This is not the case for soft keyboards, for which the current logic was designed as it requires extra processing.

Addresses #70751 for hardware keyboards (connected or bluetooth), soft keyboards still have that issue and will be addressed in a separate PR.

3.x version

When a hardware keyboard is connected, all key events come through so we can route them directly to the engine.
This is not the case for soft keyboards, for which the current logic was designed as it requires extra processing.
@m4gr3d m4gr3d force-pushed the fix_hardware_keyboard_input_routing_main branch from 36bbd14 to 87102e3 Compare August 23, 2023 16:57
@m4gr3d m4gr3d requested a review from thebestnom August 28, 2023 15:24
@YuriSizov YuriSizov merged commit 55840c5 into godotengine:master Sep 14, 2023
15 checks passed
@YuriSizov
Copy link
Contributor

YuriSizov commented Sep 14, 2023

Thanks! Would still be great to get feedback from @thebestnom in case there is more to it than meets the eye :)

@m4gr3d m4gr3d deleted the fix_hardware_keyboard_input_routing_main branch September 14, 2023 13:54
@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Oct 24, 2023
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.3.

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

Successfully merging this pull request may close these issues.

3 participants