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

Android - bug #677 and bug # 668 #57

Closed
wants to merge 5 commits into from
Closed

Android - bug #677 and bug # 668 #57

wants to merge 5 commits into from

Conversation

Pooouf
Copy link
Contributor

@Pooouf Pooouf commented Jan 28, 2013

Bug #677 : on Android, notification mechanism when the user removes the softKeyboard using the Back button
cf. https://jogamp.org/bugzilla/show_bug.cgi?id=677

Eric B.

@Pooouf
Copy link
Contributor Author

Pooouf commented Jan 28, 2013

Also added a fix proposal for https://jogamp.org/bugzilla/show_bug.cgi?id=668
That bug makes all keyboard input uppercase on Android. This is a serious matter, since it prevents some characters, like '@' for instance.

Eric Brayet added 3 commits January 29, 2013 14:58
Following the discussion and comments 2 and 3 in
the bug report, this commit proposes a better
implementation:

When the softKeyboard is visible
And the user pressed its Back button
Then WindowDriver handles a key event
VK_KEYBOARD_INVISIBLE

This is more platform agnostic than (yet another)
listener.
There is no way to know the visibility of
the softKeyboard on Android. So, we expect that
the developer uses GLWindow::setKeyboardVisible()
and store the visibility.

About comment #3 : we want the regular Back button
behavior when the softKeyboard is visible,
so dispatchKeyEventPreIme() must return false
and let the IME handle the event.
@Pooouf Pooouf closed this Feb 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant