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

Merge of "change focused widgets with arrow keys" broke l/r keys use while editing single line text fields #3324

Closed
apessino opened this issue Sep 9, 2023 · 1 comment · Fixed by #3352
Labels
bug Something is broken
Milestone

Comments

@apessino
Copy link

apessino commented Sep 9, 2023

Since #3272 was merged in, the left/right arrow keys switch focus even while a single line text editing box is focused and text is being edited. This makes it impossible to move within the text being edited without using the mouse, which is of course not expected behavior.

To verify: run egui_demo_app, click to edit a single line text field, hit the left or right arrow keys and the text edit loses focus.

I only took a quick look at the code, but it seems that single line text fields don't "focus lock" because the tab key has always been used to end their editing, but now the arrow keys follow through and also cause the focus to move away.

@apessino apessino added the bug Something is broken label Sep 9, 2023
@emilk emilk added this to the 0.23.0 milestone Sep 9, 2023
@emilk
Copy link
Owner

emilk commented Sep 10, 2023

Good catch - I'll make sure it is fixed before the next release

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

Successfully merging a pull request may close this issue.

2 participants