loleaflet/src/control/Control.LokDialog.js | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
New commits:
commit 27ad4cd17bce93dee059ed4accf5b60c8f1516c1
Author: Jan Holesovsky <ke...@collabora.com>
AuthorDate: Thu Jul 4 15:10:22 2019 +0200
Commit: Henry Castro <hcas...@collabora.com>
CommitDate: Tue Aug 20 17:11:49 2019 +0200
Fix TypeError.
No idea why window.mode.isMobile() did not work here - it does for me.
The previous code was causing a TypeError exception on the desktop.
Change-Id: I0108e2e4ac5d3ba487ac94b6459ea3f78032c606
Reviewed-on: https://gerrit.libreoffice.org/77839
Reviewed-by: Henry Castro <hcas...@collabora.com>
Tested-by: Henry Castro <hcas...@collabora.com>
diff --git a/loleaflet/src/control/Control.LokDialog.js
b/loleaflet/src/control/Control.LokDialog.js
index 00c957de5..cb4a68e5b 100644
--- a/loleaflet/src/control/Control.LokDialog.js
+++ b/loleaflet/src/control/Control.LokDialog.js
@@ -543,8 +543,9 @@ L.Control.LokDialog = L.Control.extend({
offsetY = -(height - window.screen.height) / 2;
}
- // FIXME. window.mode.isMobile() return false
- if (L.Browser.mobile() && screen.width < 768 && height <
window.screen.height) {
+ // on mobile, force the positioning to the top, so that it is
not
+ // covered by the virtual keyboard
+ if (window.mode.isMobile()) {
$(dialogContainer).dialog('option', 'position', { my:
'left top', at: 'let top', of: window });
offsetY = 0;
}
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
