kit/ChildSession.cpp                          |   15 +++++++++------
 loleaflet/src/control/Control.PartsPreview.js |    8 ++++++++
 2 files changed, 17 insertions(+), 6 deletions(-)

New commits:
commit 11558dff1a90692575a3cd2c63fa9ddbf8cd547d
Author:     Tor Lillqvist <t...@collabora.com>
AuthorDate: Fri Oct 4 15:43:17 2019 +0300
Commit:     Andras Timar <andras.ti...@collabora.com>
CommitDate: Fri Oct 18 22:31:33 2019 +0200

    tdf#127942: Make a slide that is being dragged (for re-ordering) the 
selected
    
    Otherwise we would have to add a new message to to the protocol to
    move an arbitrary part of a document. (Currently there only is the
    'moveselectedclientparts' message.) Also a new API would be needed in
    LibreOfficeKit to move an arbitrary part.
    
    Change-Id: I3012982cb0c2bc18e55f47554be51958817f9015
    Reviewed-on: https://gerrit.libreoffice.org/80214
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Tor Lillqvist <t...@collabora.com>
    (cherry picked from commit 71be6ce1ac3533df4cd04b8c01beef89888d5809)
    Reviewed-on: https://gerrit.libreoffice.org/80584
    Reviewed-by: Andras Timar <andras.ti...@collabora.com>
    Tested-by: Andras Timar <andras.ti...@collabora.com>

diff --git a/loleaflet/src/control/Control.PartsPreview.js 
b/loleaflet/src/control/Control.PartsPreview.js
index 92bbd2673..f40d4fe7a 100644
--- a/loleaflet/src/control/Control.PartsPreview.js
+++ b/loleaflet/src/control/Control.PartsPreview.js
@@ -357,6 +357,14 @@ L.Control.PartsPreview = L.Control.extend({
        },
 
        _handleDragStart: function (e) {
+               // To avoid having to add a new message to move an arbitrary 
part, let's select the
+               // slide that is being dragged.
+               var part = $('#slide-sorter .mCSB_container 
.preview-frame').index(e.target.parentNode);
+               if (part !== null) {
+                       var partId = parseInt(part) - 1; // The first part is 
just a drop-site for reordering.
+                       this.partsPreview._map.setPart(partId);
+                       this.partsPreview._map.selectPart(partId, 1, false); // 
And select.
+               }
                // By default we move when dragging, but can
                // support duplication with ctrl in the future.
                e.dataTransfer.effectAllowed = 'move';
commit 34f2a934f7d9c57c08818dd0458f1f36ff2a0188
Author:     Ashod Nakashian <ashod.nakash...@collabora.co.uk>
AuthorDate: Sun Jan 20 16:44:28 2019 -0500
Commit:     Andras Timar <andras.ti...@collabora.com>
CommitDate: Fri Oct 18 22:31:20 2019 +0200

    wsd: don't warn incorrectly when selecting current part
    
    Change-Id: I3c872469911b1f232e40cc110859402fc0814f3a
    Reviewed-on: https://gerrit.libreoffice.org/69640
    Reviewed-by: Ashod Nakashian <ashnak...@gmail.com>
    Tested-by: Ashod Nakashian <ashnak...@gmail.com>
    (cherry picked from commit 3b931560b945c25bb820ba0b24dab7d229f3336b)
    Reviewed-on: https://gerrit.libreoffice.org/80583
    Reviewed-by: Andras Timar <andras.ti...@collabora.com>
    Tested-by: Andras Timar <andras.ti...@collabora.com>

diff --git a/kit/ChildSession.cpp b/kit/ChildSession.cpp
index fdccb6ae4..6699c67a5 100644
--- a/kit/ChildSession.cpp
+++ b/kit/ChildSession.cpp
@@ -1815,14 +1815,17 @@ bool ChildSession::selectClientPart(const char* 
/*buffer*/, int /*length*/, cons
 
     getLOKitDocument()->setView(_viewId);
 
-    if (getLOKitDocument()->getDocumentType() != LOK_DOCTYPE_TEXT && nPart != 
getLOKitDocument()->getPart())
+    if (getLOKitDocument()->getDocumentType() != LOK_DOCTYPE_TEXT)
     {
-        getLOKitDocument()->selectPart(nPart, nSelect);
+        if (nPart != getLOKitDocument()->getPart())
+        {
+            getLOKitDocument()->selectPart(nPart, nSelect);
 
-        // Notify the client of the selection update.
-        const std::string status = 
LOKitHelper::documentStatus(getLOKitDocument()->get());
-        if (!status.empty())
-            return sendTextFrame("statusupdate: " + status);
+            // Notify the client of the selection update.
+            const std::string status = 
LOKitHelper::documentStatus(getLOKitDocument()->get());
+            if (!status.empty())
+                return sendTextFrame("statusupdate: " + status);
+        }
     }
     else
     {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to