loleaflet/src/control/Control.ColumnHeader.js | 2 +- loleaflet/src/control/Control.RowHeader.js | 2 +- loleaflet/src/control/Control.Toolbar.js | 14 +++++++------- loleaflet/src/control/Signing.js | 6 +++--- loleaflet/src/map/Map.js | 6 +++--- 5 files changed, 15 insertions(+), 15 deletions(-)
New commits: commit 30f37ef7b080c12febea4971a4ac5cf26a517c5c Author: Henry Castro <hcas...@collabora.com> AuthorDate: Thu Oct 10 09:26:29 2019 -0400 Commit: Henry Castro <hcas...@collabora.com> CommitDate: Thu Oct 10 15:41:56 2019 +0200 loleaflet: debug: use unique name for document container I had a not good debugging session trying to find out when the CSS properties of the document container are changing dynamically. The best scenario is to use the unique name documentContainer, so we can track all CSS properties dynamic changes, and it can set breakpoint in a suspicious change for deeper analysis. Change-Id: I3accef1c374689f04da336cd0a84047b498540b1 Reviewed-on: https://gerrit.libreoffice.org/80610 Reviewed-by: Henry Castro <hcas...@collabora.com> Tested-by: Henry Castro <hcas...@collabora.com> diff --git a/loleaflet/src/control/Control.ColumnHeader.js b/loleaflet/src/control/Control.ColumnHeader.js index d83c8303c..2b96ed317 100644 --- a/loleaflet/src/control/Control.ColumnHeader.js +++ b/loleaflet/src/control/Control.ColumnHeader.js @@ -672,7 +672,7 @@ L.Control.ColumnHeader = L.Control.Header.extend({ return; var rowHeader = L.DomUtil.get('spreadsheet-header-rows-container'); - var document = L.DomUtil.get('document-container'); + var document = this.options.documentContainer; this._setCornerCanvasHeight(height); var deltaTop = height - this._canvasHeight; diff --git a/loleaflet/src/control/Control.RowHeader.js b/loleaflet/src/control/Control.RowHeader.js index a0116122a..ab566a9a1 100644 --- a/loleaflet/src/control/Control.RowHeader.js +++ b/loleaflet/src/control/Control.RowHeader.js @@ -635,7 +635,7 @@ L.Control.RowHeader = L.Control.Header.extend({ return; var columnHeader = L.DomUtil.get('spreadsheet-header-columns-container'); - var document = L.DomUtil.get('document-container'); + var document = this.options.documentContainer; this._setCornerCanvasWidth(width); diff --git a/loleaflet/src/control/Control.Toolbar.js b/loleaflet/src/control/Control.Toolbar.js index fd018929e..7521180db 100644 --- a/loleaflet/src/control/Control.Toolbar.js +++ b/loleaflet/src/control/Control.Toolbar.js @@ -1577,7 +1577,7 @@ function onDocLayerInit() { if (!window.ThisIsTheiOSApp && window.mode.isTablet()) { map.hideMenubar(); - $('#document-container').addClass('tablet'); + $(map.options.documentContainer).addClass('tablet'); $('#spreadsheet-row-column-frame').addClass('tablet'); $('#tb_editbar_item_fonts').css({'display': 'none'}); @@ -2113,15 +2113,15 @@ function onUpdatePermission(e) { $('#toolbar-down').show(); switch (map._docLayer._docType) { case 'text': - $('#document-container').css('bottom', '35px'); + $(map.options.documentContainer).css('bottom', '35px'); break; case 'spreadsheet': - $('#document-container').css('bottom', '68px'); + $(map.options.documentContainer).css('bottom', '68px'); $('#spreadsheet-row-column-frame').css('bottom', '68px'); $('#spreadsheet-toolbar').show(); break; case 'presentation': - $('#document-container').css('bottom', '95px'); + $(map.options.documentContainer).css('bottom', '95px'); break; } } @@ -2170,15 +2170,15 @@ function onUpdatePermission(e) { $('#toolbar-down').hide(); switch (map._docLayer._docType) { case 'text': - $('#document-container').css('bottom', '0px'); + $(map.options.documentContainer).css('bottom', '0px'); break; case 'spreadsheet': - $('#document-container').css('bottom', '35px'); + $(map.options.documentContainer).css('bottom', '35px'); $('#spreadsheet-row-column-frame').css('bottom', '35px'); $('#spreadsheet-toolbar').show(); break; case 'presentation': - $('#document-container').css('bottom', '0px'); + $(map.options.documentContainer).css('bottom', '0px'); break; } } diff --git a/loleaflet/src/control/Signing.js b/loleaflet/src/control/Signing.js index 4cb539bd1..4489f325f 100644 --- a/loleaflet/src/control/Signing.js +++ b/loleaflet/src/control/Signing.js @@ -437,9 +437,9 @@ L.Map.include({ showSignDocument: function() { $('#document-signing-bar').show(); this.initializeLibrary(); - oldtoolbarSize = $('#document-container').css('top'); + oldtoolbarSize = $(this.options.documentContainer).css('top'); - $('#document-container').css('top', '116px'); + $(this.options.documentContainer).css('top', '116px'); // Avoid scroll button ">>" var el = w2ui['document-signing-bar']; @@ -451,7 +451,7 @@ L.Map.include({ library = null; identity = null; currentPassport = null; - $('#document-container').css('top', oldtoolbarSize); + $(this.options.documentContainer).css('top', oldtoolbarSize); }, signingInitializeBar: function() { $('#document-signing-bar').hide(); diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js index 89c80ff98..20e948353 100644 --- a/loleaflet/src/map/Map.js +++ b/loleaflet/src/map/Map.js @@ -148,7 +148,7 @@ L.Map = L.Evented.extend({ // Avoid white bar on the bottom - force resize-detector to get full size if (window.mode.isMobile()) { - $('#document-container').css('bottom', '0px'); + $(this.options.documentContainer).css('bottom', '0px'); $(this._resizeDetector).css('bottom', '0px'); } @@ -1678,7 +1678,7 @@ L.Map = L.Evented.extend({ obj.addClass('fold'); moveObjectVertically($('#spreadsheet-row-column-frame'), 36); - moveObjectVertically($('#document-container'), 36); + moveObjectVertically($(this.options.documentContainer), 36); moveObjectVertically($('#presentation-controls-wrapper'), 36); moveObjectVertically($('#sidebar-dock-wrapper'), 36); }, @@ -1696,7 +1696,7 @@ L.Map = L.Evented.extend({ obj.addClass('unfold'); moveObjectVertically($('#spreadsheet-row-column-frame'), -36); - moveObjectVertically($('#document-container'), -36); + moveObjectVertically($(this.options.documentContainer), -36); moveObjectVertically($('#presentation-controls-wrapper'), -36); moveObjectVertically($('#sidebar-dock-wrapper'), -36); }, _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits