loleaflet/src/control/Control.Toolbar.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
New commits: commit 799fee972b61ecaf10fb70a5dc999dfa293838c5 Author: Tor Lillqvist <t...@collabora.com> AuthorDate: Mon Mar 30 12:42:54 2020 +0300 Commit: Tor Lillqvist <t...@collabora.com> CommitDate: Mon Mar 30 12:12:41 2020 +0200 Clarify meaning of the mobile and tablet properties in the toolItems elements In particular, also here "mobile" actually means "mobile phone" and not a mobile devie in general. Change-Id: If3bdf0ae9fa2da43364413ab08387053d67121b2 Reviewed-on: https://gerrit.libreoffice.org/c/online/+/91351 Tested-by: Tor Lillqvist <t...@collabora.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 c09f4a427..059dc28e5 100644 --- a/loleaflet/src/control/Control.Toolbar.js +++ b/loleaflet/src/control/Control.Toolbar.js @@ -812,9 +812,6 @@ function hideTooltip(toolbar, id) { var stylesSelectValue; var fontsSelectValue; -// mobile:false means hide it both for normal Online used from a mobile browser, and in a mobile app -// mobilebrowser:false means hide it for normal Online used from a mobile browser, but don't hide it in a mobile app - function createToolbar() { if (window.mode.isMobile()) { @@ -826,6 +823,11 @@ function createToolbar() { } } +// mobile:false means hide it both for normal Online used from a mobile phone browser, and in a mobile app on a mobile phone +// mobilebrowser:false means hide it for normal Online used from a mobile browser, but don't hide it in a mobile app +// tablet:true means show it in normal Online from a tablet browser, and in a mobile app on a tablet +// tablet:false means hide it in normal Online used from a tablet browser, and in a mobile app on a tablet + function initNormalToolbar() { var toolItems = [ {type: 'button', id: 'closemobile', img: 'closemobile', desktop: false, mobile: false, tablet: true, hidden: true}, _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits