[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - sc/source

[Dennis Francis (via logerrit)]

 sc/source/ui/view/viewdata.cxx |   14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

New commits:
commit d88cad2c190d0cc43ee6aaaaf82396be8fdf15fc
Author:     Dennis Francis <dennis.francis at collabora.com>
AuthorDate: Sun May 24 17:21:59 2020 +0530
Commit:     Dennis Francis <dennis.francis at collabora.com>
CommitDate: Sat Jul 4 19:23:26 2020 +0200

    lokit: do not set an artificial cursor size when it is zero...
    
    when we send the message in print twips coordinates. It is important for
    the client to know the exact position and coordinates to allow client
    side print-twips -> tile-twips conversion when/if it is needed.
    
    Change-Id: I6699894758886f1b5648ac9bf3c9e6ab4192c72e
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97963
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Dennis Francis <dennis.francis at collabora.com>

diff --git a/sc/source/ui/view/viewdata.cxx b/sc/source/ui/view/viewdata.cxx
index 5b5b4baf3b83..2393f27a27ae 100644
--- a/sc/source/ui/view/viewdata.cxx
+++ b/sc/source/ui/view/viewdata.cxx
@@ -2257,15 +2257,13 @@ OString ScViewData::describeCellCursorAt(SCCOL nX, SCROW nY, bool bPixelAligned)
     }
     else
     {
-        // make it a slim cell cursor, but not empty
-        if (nSizeX == 0)
-            nSizeX = TWIPS_PER_PIXEL;
-        if (nSizeY == 0)
-            nSizeY = TWIPS_PER_PIXEL;
-
+        // look at Rectangle( const Point& rLT, const Size& rSize ) for the decrement.
+        if (nSizeX)
+            --nSizeX;
+        if (nSizeY)
+            --nSizeY;
         ss << aCellPos.getX() << ", " << aCellPos.getY()
-            // look at Rectangle( const Point& rLT, const Size& rSize ) for the '- 1'
-            << ", " << nSizeX - 1 << ", " << nSizeY - 1 << ", "
+            << ", " << nSizeX << ", " << nSizeY << ", "
             << nX << ", " << nY;
     }