loleaflet/dist/loleaflet.css | 4 ++++ loleaflet/dist/partsPreviewControl.css | 2 ++ loleaflet/src/control/Control.PartsPreview.js | 4 ++-- loleaflet/src/layer/tile/ImpressTileLayer.js | 11 ++++++++++- 4 files changed, 18 insertions(+), 3 deletions(-)
New commits: commit 064b2508a7bde1e2d3422824cd06bf0f7c915061 Author: Pranav Kant <pran...@collabora.co.uk> Date: Mon Mar 20 13:58:33 2017 +0530 loleaflet: Slide sorter for draw documents Change-Id: I4ad11236c0186561b5c7df1a3c634e54f8576c8c diff --git a/loleaflet/dist/loleaflet.css b/loleaflet/dist/loleaflet.css index fbecbf7e..a3880d46 100644 --- a/loleaflet/dist/loleaflet.css +++ b/loleaflet/dist/loleaflet.css @@ -50,6 +50,10 @@ body { display: block; } +#presentation-controls-wrapper.drawing { + bottom: 34px; /* Hide the presentation toolbar (insert slide etc.) for doctype = drawing */ +} + #presentation-controls-wrapper.readonly { top: 30px; } diff --git a/loleaflet/dist/partsPreviewControl.css b/loleaflet/dist/partsPreviewControl.css index 4dc87f95..438b137d 100644 --- a/loleaflet/dist/partsPreviewControl.css +++ b/loleaflet/dist/partsPreviewControl.css @@ -19,6 +19,8 @@ } .preview-img { + /* In draw docs, the width of previews are small, but we want a min of 180px to align it with document's left edge */ + min-width: 180px; vertical-align: middle; max-width: 184px; cursor: pointer; diff --git a/loleaflet/src/control/Control.PartsPreview.js b/loleaflet/src/control/Control.PartsPreview.js index c4409105..f466be8c 100644 --- a/loleaflet/src/control/Control.PartsPreview.js +++ b/loleaflet/src/control/Control.PartsPreview.js @@ -28,7 +28,7 @@ L.Control.PartsPreview = L.Control.extend({ return; } - if (docType === 'presentation') { + if (docType === 'presentation' || docType === 'drawing') { if (!this._previewInitialized) { // make room for the preview @@ -145,7 +145,7 @@ L.Control.PartsPreview = L.Control.extend({ }, _updatePreview: function (e) { - if (this._map.getDocType() === 'presentation') { + if (this._map.getDocType() === 'presentation' || this._map.getDocType() === 'drawing') { // the scrollbar has to be re-initialized here else it doesn't work // probably a bug from the scrollbar this._previewTiles[e.id].onload = function () { diff --git a/loleaflet/src/layer/tile/ImpressTileLayer.js b/loleaflet/src/layer/tile/ImpressTileLayer.js index 2584679d..1390216e 100644 --- a/loleaflet/src/layer/tile/ImpressTileLayer.js +++ b/loleaflet/src/layer/tile/ImpressTileLayer.js @@ -145,7 +145,13 @@ L.ImpressTileLayer = L.TileLayer.extend({ }, _onCommandValuesMsg: function (textMsg) { - var values = JSON.parse(textMsg.substring(textMsg.indexOf('{'))); + try { + var values = JSON.parse(textMsg.substring(textMsg.indexOf('{'))); + } catch (e) { + // One such case is 'commandvalues: ' for draw documents in response to .uno:AcceptTrackedChanges + values = null; + } + if (!values) { return; } @@ -325,6 +331,9 @@ L.ImpressTileLayer = L.TileLayer.extend({ this._docWidthTwips = command.width; this._docHeightTwips = command.height; this._docType = command.type; + if (this._docType === 'drawing') { + L.DomUtil.addClass(L.DomUtil.get('presentation-controls-wrapper'), 'drawing'); + } this._updateMaxBounds(true); this._documentInfo = textMsg; this._parts = command.parts; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits