loleaflet/css/device-mobile.css               |   34 +++++++++++++++++++++++
 loleaflet/css/partsPreviewControl.css         |   38 --------------------------
 loleaflet/src/control/Control.PartsPreview.js |   21 ++++++++++++++
 loleaflet/src/layer/tile/ImpressTileLayer.js  |    6 ++--
 4 files changed, 59 insertions(+), 40 deletions(-)

New commits:
commit 7cd2beb8660498d1dedd37b23057c4e4b0bc55a4
Author:     Jan Holesovsky <ke...@collabora.com>
AuthorDate: Thu Apr 2 14:14:19 2020 +0200
Commit:     Jan Holesovsky <ke...@collabora.com>
CommitDate: Thu Apr 2 16:37:45 2020 +0200

    mobile: Handle the portrait / landscape for slide sorter in JS too.
    
    Change-Id: I214501fac516f4d581d3c1e8a95650a87db1a4aa
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/91570
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Jan Holesovsky <ke...@collabora.com>

diff --git a/loleaflet/css/device-mobile.css b/loleaflet/css/device-mobile.css
index 5a40ba1b2..685d16454 100644
--- a/loleaflet/css/device-mobile.css
+++ b/loleaflet/css/device-mobile.css
@@ -624,6 +624,24 @@ td[id^=tb_spreadsheet-toolbar_item]:focus 
table.w2ui-button div.w2ui-icon, td[id
        bottom: 33px;
 }
 
+#slide-sorter.portrait {
+       max-height: 60px;
+}
+
+.preview-img-portrait {
+       min-width: 37px;
+       max-width: 60px;
+       max-height: 45px;
+       margin: 0px;
+}
+
+.preview-frame-portrait {
+       max-height: 60px;
+       max-width: initial;
+       display: table-cell;
+       padding-right: 1em;
+}
+
 /* Slidesorter in landscape mode */
 #presentation-controls-wrapper.landscape {
        top: 41px;
@@ -650,3 +668,19 @@ td[id^=tb_spreadsheet-toolbar_item]:focus 
table.w2ui-button div.w2ui-icon, td[id
 #document-container.landscape.parts-preview-document.keyboard {
        left: 0px !important;
 }
+
+#slide-sorter.landscape {
+       max-width: 120px;
+}
+
+.preview-img-landscape {
+       min-width: 40px;
+       max-width: 40px;
+       margin-left: 8px;
+}
+
+.preview-frame-landscape {
+       max-height: initial;
+       max-width: 66px;
+       display: block;
+}
diff --git a/loleaflet/css/partsPreviewControl.css 
b/loleaflet/css/partsPreviewControl.css
index 618d2e0a7..4ea0d3c7e 100644
--- a/loleaflet/css/partsPreviewControl.css
+++ b/loleaflet/css/partsPreviewControl.css
@@ -33,44 +33,6 @@
        margin-left: 20px;
        }
 
-@media (max-width: 767px) and (orientation: portrait), (max-device-width: 
767px) and (orientation: portrait) {
-       #slide-sorter {
-               max-height: 60px;
-       }
-
-       .preview-img {
-               min-width: 37px;
-               max-width: 60px;
-               max-height: 45px;
-               margin: 0px;
-       }
-
-       .preview-frame {
-               max-height: 60px;
-               max-width: initial;
-               display: table-cell;
-               padding-right: 1em;
-       }
-}
-
-@media (max-height: 767px) and (orientation: landscape), (max-device-height: 
767px) and (orientation: landscape) {
-       #slide-sorter {
-               max-width: 120px;
-       }
-
-       .preview-img {
-               min-width: 40px;
-               max-width: 40px;
-               margin-left: 8px;
-       }
-
-       .preview-frame {
-               max-height: initial;
-               max-width: 66px;
-               display: block;
-       }
-}
-
 /* The current part the user is on. */
 .preview-img-currentpart {
     border-color: #000000;
diff --git a/loleaflet/src/control/Control.PartsPreview.js 
b/loleaflet/src/control/Control.PartsPreview.js
index 4d9776566..22c81be19 100644
--- a/loleaflet/src/control/Control.PartsPreview.js
+++ b/loleaflet/src/control/Control.PartsPreview.js
@@ -150,6 +150,27 @@ L.Control.PartsPreview = L.Control.extend({
                                                
L.DomUtil.addClass(this._previewTiles[j], 'preview-img-selectedpart');
                                }
                        }
+
+                       // update portrait / landscape
+                       var removePreviewImg = 'preview-img-portrait';
+                       var addPreviewImg = 'preview-img-landscape';
+                       var removePreviewFrame = 'preview-frame-portrait';
+                       var addPreviewFrame = 'preview-frame-landscape';
+                       if (L.DomUtil.isPortrait()) {
+                               removePreviewImg = 'preview-img-landscape';
+                               addPreviewImg = 'preview-img-portrait';
+                               removePreviewFrame = 'preview-frame-landscape';
+                               addPreviewFrame = 'preview-frame-portrait';
+                       }
+
+                       for (i = 0; i < parts; i++) {
+                               L.DomUtil.removeClass(this._previewTiles[i], 
removePreviewImg);
+                               L.DomUtil.addClass(this._previewTiles[i], 
addPreviewImg);
+                       }
+
+                       var previewFrame = 
$(this._partsPreviewCont).find('.preview-frame');
+                       previewFrame.removeClass(removePreviewFrame);
+                       previewFrame.addClass(addPreviewFrame);
                }
        },
 
diff --git a/loleaflet/src/layer/tile/ImpressTileLayer.js 
b/loleaflet/src/layer/tile/ImpressTileLayer.js
index 0319cf7b1..ee0b75e9c 100644
--- a/loleaflet/src/layer/tile/ImpressTileLayer.js
+++ b/loleaflet/src/layer/tile/ImpressTileLayer.js
@@ -100,6 +100,7 @@ L.ImpressTileLayer = L.TileLayer.extend({
 
                var presentationControlWrapperElem = 
L.DomUtil.get('presentation-controls-wrapper');
                var documentContainer = L.DomUtil.get('document-container');
+               var slideSorter = L.DomUtil.get('slide-sorter');
 
                // update portrait / landscape
                var remove = 'portrait';
@@ -111,12 +112,14 @@ L.ImpressTileLayer = L.TileLayer.extend({
 
                L.DomUtil.removeClass(presentationControlWrapperElem, remove);
                L.DomUtil.removeClass(documentContainer, remove);
+               L.DomUtil.removeClass(slideSorter, remove);
                L.DomUtil.addClass(presentationControlWrapperElem, add);
                L.DomUtil.addClass(documentContainer, add);
+               L.DomUtil.addClass(slideSorter, add);
 
                // update parts
                var visible = 
L.DomUtil.getStyle(presentationControlWrapperElem, 'display');
-               if (!this._isSlidePaneVisible && visible !== 'none') {
+               if (visible !== 'none') {
                        this._map.fire('updateparts', {
                                selectedPart: this._selectedPart,
                                selectedParts: this._selectedParts,
@@ -125,7 +128,6 @@ L.ImpressTileLayer = L.TileLayer.extend({
                                partNames: this._partHashes
                        });
                }
-               this._isSlidePaneVisible = !(visible === 'none');
        },
 
        onMobileInit: function (map) {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to