erabo/spiegel-keyman
1
0
Fork 0
mirror of https://github.com/keymanapp/keyman.git synced 2026-08-05 00:15:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

fix(android): handle blank keyboard tap properly

Browse source 
Previously, the event handler was attached to body, but in the event of
the keyboard failing to render, the body element will typically be zero
pixels high. Instead, attach the handler to the document itself.
KeymanWeb, when running normally, will handle all touches on the
keyboard elements which take 100% of the real estate. So the document
tap will only be received in the case of an error condition.

Also, add a console error that will be reported to Sentry when user has
error reporting switched on.

Test-bot: skip
This commit is contained in:
Marc Durdin 2026-07-10 14:08:27 +02:00
parent 00c213ba4e
commit 2b078ce165
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
android/KMEA/app/src/main/assets/android-host.js
View file

@ -22,10 +22,6 @@ sentryManager.init();
window.addEventListener('load', init, false);
function loadDefaultKeyboard() {
notifyHost('reloadAfterError');
}
function init() {
//document.body.style.backgroundColor="transparent";
//window.console.log('Device type = '+device);
@ -82,7 +78,17 @@ function init() {
keyman.addEventListener('keyboardchange', setIsChiral);
keyman.core.languageProcessor.on('statechange', onStateChange);
document.body.addEventListener('touchend', loadDefaultKeyboard);
// If the keyboard fails to display, this usually indicates that a Javascript
// error has occurred, perhaps in the active keyboard. So we have a last-gasp
// fallback in that situation; if the user touches the blank document, we will
// attempt to reload the default keyboard.
document.addEventListener('touchend', () => {
// The error will be reported through to Sentry if the user has error
// reporting enabled; breadcrumbs can give us helpful details on possible
// root causes for the error
window.console.error('user touched the document body, which can only happen if the keyboard was not presented. This is usually caused by another error.');
notifyHost('reloadAfterError');
});
notifyHost('pageLoaded');
}