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 Back button terminates app after SDL_StartTextInput #3483

Closed
SDLBugzilla opened this issue Feb 11, 2021 · 0 comments
Closed

Pressing Back button terminates app after SDL_StartTextInput #3483

SDLBugzilla opened this issue Feb 11, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

Reported in version: HG 2.0
Reported for operating system, platform: Android (All), ARM

Comments on the original bug report:

On 2019-12-19 05:31:26 +0000, Miku AuahDark wrote:

Hello, this is my first time filling a bug.

I noticed a problem when I call function to show the on-screen keyboard and hide it again, I didn't receive "Back" button key event (keyup/down) and the app exit instead (but not crash as I receive the quit event). If I set SDL_ANDROID_TRAP_BACK_BUTTON hint, the app didn't exit but I didn't receive "Back" button key event either.

This is similar to # 4702 but the difference is that I only encounter this issue AFTER calling "SDL_StartTextInput". This means that if I didn't call this function, I receive "Back" button key event correctly, even without the hint described above. As I'm testing this with changeset 472db3f71645 (tip as of writing), this means the fix there isn't effective to me.

My device is Mi A1 running Android 9 unrooted. I can add traces to the SDL code if needed.

On 2019-12-19 12:54:05 +0000, Sylvain wrote:

I think this is fixed with:
https://hg.libsdl.org/SDL/rev/381ddcd3957c

When the text input is hidden, we need to set back the focus on mSurface
Please let me know

On 2019-12-19 14:44:10 +0000, Miku AuahDark wrote:

Didn't know the fix is that simple. Thanks, it now properly capture "Back" button as escape again.

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

No branches or pull requests

1 participant