sc/source/ui/view/gridwin4.cxx |   12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

New commits:
commit 142e56c21f711a8d5b015d964a7db7a9211a1294
Author:     user <szymon.k...@collabora.com>
AuthorDate: Thu Nov 10 21:27:36 2022 +0100
Commit:     Gökay ŞATIR <gokaysa...@collabora.com>
CommitDate: Mon Nov 14 09:11:22 2022 +0100

    Revert "lok: respect zoom in calc tile rendering of EditView"
    
    This reverts commit 7260b8724fafd41294761382f69ece7fe09588c4.
    lok: respect zoom in calc tile rendering of EditView
    
    Which was causing chart artifacts. Original issue which was
    fixed by above change is no longer valid. Probably different
    fix was made.
    
    Change-Id: I855be90b2736a2d1e95dc1cabef54a438644aa7d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142570
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Gökay ŞATIR <gokaysa...@collabora.com>

diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx
index 970a4591411f..6fdd332092de 100644
--- a/sc/source/ui/view/gridwin4.cxx
+++ b/sc/source/ui/view/gridwin4.cxx
@@ -749,8 +749,7 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice, const 
ScTableInfo& rTableI
         // the same as editeng and drawinglayer), and get rid of all the
         // SetMapMode's and other unnecessary fun we have with pixels
         // See also ScGridWindow::GetDrawMapMode() for the rest of this hack
-        aDrawMode.SetOrigin(PixelToLogic(Point(tools::Long(nScrX / 
aOutputData.aZoomX),
-                                               tools::Long(nScrY / 
aOutputData.aZoomY)), aDrawMode));
+        aDrawMode.SetOrigin(PixelToLogic(Point(nScrX, nScrY), aDrawMode));
     }
     tools::Rectangle aDrawingRectLogic;
     bool bLayoutRTL = rDoc.IsLayoutRTL( nTab );
@@ -1055,9 +1054,6 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice, 
const ScTableInfo& rTableI
 
             if (bIsTiledRendering)
             {
-                const double fZoomX = static_cast<double>(aOutputData.aZoomX);
-                const double fZoomY = static_cast<double>(aOutputData.aZoomY);
-
                 Point aOrigin = aOriginalMode.GetOrigin();
                 if (bLayoutRTL)
                     aOrigin.setX(-o3tl::convert(aOrigin.getX(), 
o3tl::Length::twip, o3tl::Length::px)
@@ -1069,8 +1065,8 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice, 
const ScTableInfo& rTableI
                 aOrigin.setY(o3tl::convert(aOrigin.getY(), o3tl::Length::twip, 
o3tl::Length::px)
                              + aOutputData.nScrY);
                 const double twipFactor = 15 * 1.76388889; // 26.45833335
-                aOrigin = Point(aOrigin.getX() * twipFactor / fZoomX,
-                                aOrigin.getY() * twipFactor / fZoomY);
+                aOrigin = Point(aOrigin.getX() * twipFactor,
+                                aOrigin.getY() * twipFactor);
                 MapMode aNew = rDevice.GetMapMode();
                 aNew.SetOrigin(aOrigin);
                 rDevice.SetMapMode(aNew);
@@ -1657,7 +1653,7 @@ void ScGridWindow::PaintTile( VirtualDevice& rDevice,
                              -nTopLeftTileRowOffset,
                              nTopLeftTileCol, nTopLeftTileRow,
                              nBottomRightTileCol, nBottomRightTileRow,
-                             fPPTX, fPPTY, &aFracX, &aFracY);
+                             fPPTX, fPPTY, nullptr, nullptr);
 
     // setup the SdrPage so that drawinglayer works correctly
     ScDrawLayer* pModel = rDoc.GetDrawLayer();

Reply via email to