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

Reply via email to