loleaflet/src/control/Control.Menubar.js   |    7 ++++---
 loleaflet/src/control/Control.StatusBar.js |    3 ++-
 2 files changed, 6 insertions(+), 4 deletions(-)

New commits:
commit 70f4f8b92e44d361c78dd323e147d23abba8b308
Author:     Tor Lillqvist <t...@collabora.com>
AuthorDate: Thu Apr 16 08:09:44 2020 +0300
Commit:     Tor Lillqvist <t...@collabora.com>
CommitDate: Thu Apr 16 09:26:01 2020 +0200

    Don't show zoom controls on tablets, either
    
    On touch devices you use pinch gestures to zoom, not buttons or menus.
    Displaying the alleged zoom percentage is fairly pointless as it
    doesn't match physical paper size anyway.
    
    Change-Id: Ieccfa9b5454a3ac71b0c31f6ab38ed6696304cc5
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92332
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Tor Lillqvist <t...@collabora.com>

diff --git a/loleaflet/src/control/Control.Menubar.js 
b/loleaflet/src/control/Control.Menubar.js
index 919a4ab3f..8d77a6dc3 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -56,7 +56,8 @@ L.Control.Menubar = L.Control.extend({
                                ]},
                                {uno: '.uno:EditStyle'}
                        ]},
-                       {name: _UNO('.uno:ViewMenu', 'text'), id: 'view', type: 
'menu', menu: [
+                       {name: _UNO('.uno:ViewMenu', 'text'), id: 'view', type: 
'menu',
+                        menu: (window.mode.isTablet() ? [] : [
                                {name: _UNO('.uno:FullScreen', 'text'), id: 
'fullscreen', type: 'action'},
                                {type: 'separator'},
                                {name: _UNO('.uno:ZoomPlus', 'text'), id: 
'zoomin', type: 'action'},
@@ -64,13 +65,13 @@ L.Control.Menubar = L.Control.extend({
                                {name: _('Reset zoom'), id: 'zoomreset', type: 
'action'},
                                {name: _('Show Ruler'), id: 'showruler', type: 
'action'},
                                {type: 'separator'},
+                       ]).concat([
                                {uno: '.uno:ControlCodes'},
                                {type: 'separator'},
                                {name: _UNO('.uno:ShowResolvedAnnotations', 
'text'), id: 'showresolved', type: 'action'},
                                {type: 'separator'},
                                {uno: '.uno:Sidebar'},
-                       ]
-                       },
+                       ])},
                        {name: _UNO('.uno:InsertMenu', 'text'), id: 'insert', 
type: 'menu', menu: [
                                {name: _('Local Image...'), id: 
'insertgraphic', type: 'action'},
                                {name: _UNO('.uno:InsertGraphic', 'text'), id: 
'insertgraphicremote', type: 'action'},
diff --git a/loleaflet/src/control/Control.StatusBar.js 
b/loleaflet/src/control/Control.StatusBar.js
index 26db62d11..ed5eb0353 100644
--- a/loleaflet/src/control/Control.StatusBar.js
+++ b/loleaflet/src/control/Control.StatusBar.js
@@ -186,6 +186,7 @@ L.Control.StatusBar = L.Control.extend({
                                        {type: 'button',  id: 'prev', img: 
'prev', hint: _UNO('.uno:PageUp', 'text')},
                                        {type: 'button',  id: 'next', img: 
'next', hint: _UNO('.uno:PageDown', 'text')},
                                        {type: 'break', id: 'prevnextbreak'},
+                               ].concat(window.mode.isTablet() ? [] : [
                                        {type: 'button',  id: 'zoomreset', img: 
'zoomreset', hint: _('Reset zoom')},
                                        {type: 'button',  id: 'zoomout', img: 
'zoomout', hint: _UNO('.uno:ZoomMinus')},
                                        {type: 'menu-radio', id: 'zoom', text: 
'100',
@@ -210,7 +211,7 @@ L.Control.StatusBar = L.Control.extend({
                                                ]
                                        },
                                        {type: 'button',  id: 'zoomin', img: 
'zoomin', hint: _UNO('.uno:ZoomPlus')}
-                               ],
+                               ]),
                                onClick: function (e) {
                                        that.hideTooltip(this, e.target);
                                        that.onClick(e, e.target, e.item, 
e.subItem);
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to