loleaflet/src/control/Control.Toolbar.js |    9 +++++++++
 1 file changed, 9 insertions(+)

New commits:
commit 32f33328d55fed1abfb0533ecee8496848931ded
Author:     Tor Lillqvist <t...@collabora.com>
AuthorDate: Fri Sep 25 12:36:47 2020 +0300
Commit:     Tor Lillqvist <t...@collabora.com>
CommitDate: Fri Sep 25 11:59:56 2020 +0200

    tdf#136457: Prevent tapping other UI elements after tapping the closebutton
    
    Change-Id: I925cd2d44f40ecca23fb51cbe2d2c6d15ecf9648
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/103377
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Tor Lillqvist <t...@collabora.com>

diff --git a/loleaflet/src/control/Control.Toolbar.js 
b/loleaflet/src/control/Control.Toolbar.js
index 3562e57fb..4aae10784 100644
--- a/loleaflet/src/control/Control.Toolbar.js
+++ b/loleaflet/src/control/Control.Toolbar.js
@@ -43,6 +43,15 @@ function getUNOCommand(unoData) {
 
 function onClose() {
        if (window.ThisIsAMobileApp) {
+               // Hide stuff to avoid the user (even accidentally) tapping 
anything else right
+               // after tapping the closebutton, before the app has torn down 
the WebView.
+               // This is a silly workaround to avoid one possible way to 
reproduce tdf#136457.
+               $('#toolbar-wrapper').hide();
+               $('#closebuttonwrapper').hide();
+               $('#main-menu').hide();
+               $('#document-titlebar').hide();
+
+               // Then tell the app to delete the web view
                window.postMobileMessage('BYE');
        } else {
                map.fire('postMessage', {msgId: 'close', args: {EverModified: 
map._everModified, Deprecated: true}});
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to