loleaflet/css/loleaflet.css | 24 ++++++++++++++++++------ loleaflet/src/control/Control.Toolbar.js | 22 ---------------------- loleaflet/src/layer/tile/CalcTileLayer.js | 1 + loleaflet/src/layer/tile/ImpressTileLayer.js | 2 -- loleaflet/src/map/Map.js | 3 --- 5 files changed, 19 insertions(+), 33 deletions(-)
New commits: commit d8833d39d3422233e97f84828bb03a544dc922fa Author: Henry Castro <hcas...@collabora.com> AuthorDate: Fri Nov 1 19:42:55 2019 -0400 Commit: Henry Castro <hcas...@collabora.com> CommitDate: Mon Nov 4 22:25:50 2019 +0100 loleaflet: mobile: avoid hardcoding CSS bottom property Use the CSS class selector to modify the bottom properties for each tile layer document, so adjusting values for portrait or landscape orientation not be so complicated. Change-Id: I900ef5c4ec3f770fd802a23238fdd9e6eb838772 Reviewed-on: https://gerrit.libreoffice.org/81928 Reviewed-by: Henry Castro <hcas...@collabora.com> Tested-by: Henry Castro <hcas...@collabora.com> diff --git a/loleaflet/css/loleaflet.css b/loleaflet/css/loleaflet.css index f53b3c618..a60a97690 100644 --- a/loleaflet/css/loleaflet.css +++ b/loleaflet/css/loleaflet.css @@ -221,34 +221,46 @@ body { left: 0px !important; } - #document-container.tablet { + #document-container.readonly { top: 41px; + bottom: 0px; } - #document-container.readonly { - top: 41px; + #document-container.readonly.parts-preview-document { + bottom: 65px; + } + + #document-container.readonly.spreadsheet-document { + top: 61px !important; + bottom: 35px; } #document-container.parts-preview-document { left: 0px !important; bottom: 95px; } + #document-container.spreadsheet-document { top: 97px !important; + bottom: 68px; } - #document-container.spreadsheet-document.readonly { - top: 61px !important; - } + #document-container { top: 41px; right: 0px !important; + bottom: 35px; } + #spreadsheet-row-column-frame { top: 77px !important; + bottom: 68px; } + #spreadsheet-row-column-frame.readonly { top: 41px !important; + bottom: 35px !important; } + #toolbar-hamburger { width: 36px; } diff --git a/loleaflet/src/control/Control.Toolbar.js b/loleaflet/src/control/Control.Toolbar.js index bac3bab0a..ba4e777aa 100644 --- a/loleaflet/src/control/Control.Toolbar.js +++ b/loleaflet/src/control/Control.Toolbar.js @@ -2138,19 +2138,8 @@ function onUpdatePermission(e) { } $('#search-input').prop('disabled', false); - // FIXME avoid hardcoding this stuff if possible if (_inMobileMode()) { $('#toolbar-down').show(); - switch (map._docLayer._docType) { - case 'text': - $(map.options.documentContainer).css('bottom', '35px'); - break; - case 'spreadsheet': - $(map.options.documentContainer).css('bottom', '68px'); - $('#spreadsheet-row-column-frame').css('bottom', '68px'); - $('#spreadsheet-toolbar').show(); - break; - } } } else { @@ -2192,19 +2181,8 @@ function onUpdatePermission(e) { } $('#search-input').prop('disabled', true); - // FIXME avoid hardcoding this stuff if possible if (_inMobileMode()) { $('#toolbar-down').hide(); - switch (map._docLayer._docType) { - case 'text': - $(map.options.documentContainer).css('bottom', '0px'); - break; - case 'spreadsheet': - $(map.options.documentContainer).css('bottom', '35px'); - $('#spreadsheet-row-column-frame').css('bottom', '35px'); - $('#spreadsheet-toolbar').show(); - break; - } } } } diff --git a/loleaflet/src/layer/tile/CalcTileLayer.js b/loleaflet/src/layer/tile/CalcTileLayer.js index 76149cf82..b5ee7ca8d 100644 --- a/loleaflet/src/layer/tile/CalcTileLayer.js +++ b/loleaflet/src/layer/tile/CalcTileLayer.js @@ -214,6 +214,7 @@ L.CalcTileLayer = L.TileLayer.extend({ L.DomEvent.preventDefault(e); } }); + toolbar.show(); toolbar = $('#toolbar-down'); toolbar.w2toolbar({ diff --git a/loleaflet/src/layer/tile/ImpressTileLayer.js b/loleaflet/src/layer/tile/ImpressTileLayer.js index ce3f727ec..b5e74a5ac 100644 --- a/loleaflet/src/layer/tile/ImpressTileLayer.js +++ b/loleaflet/src/layer/tile/ImpressTileLayer.js @@ -418,10 +418,8 @@ L.ImpressTileLayer = L.TileLayer.extend({ onUpdatePermission: function (e) { if (window.mode.isMobile()) { if (e.perm === 'edit') { - $(this._map.options.documentContainer).css('bottom', '95px'); this._addButton.addTo(this._map); } else { - $(this._map.options.documentContainer).css('bottom', '60px'); this._addButton.remove(); } } diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js index f333fbe42..1c1213aa3 100644 --- a/loleaflet/src/map/Map.js +++ b/loleaflet/src/map/Map.js @@ -147,10 +147,7 @@ L.Map = L.Evented.extend({ this._clipboardContainer = L.clipboardContainer(); this.addLayer(this._clipboardContainer); - // Avoid white bar on the bottom - force resize-detector to get full size if (window.mode.isMobile()) { - $(this.options.documentContainer).css('bottom', '0px'); - $(this._resizeDetector).css('bottom', '0px'); L.DomEvent.on(window, 'resize', function(e) { this.fire('orientationchange', e); }, this); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits