commit be32a6390d0fc4ebf60d553b31e402ca9fbdec5d
author Dennis Francis <dennis.francis@collabora.com> Tue Jan 21 22:11:19 2020 +0530
committer Dennis Francis <dennis.francis@collabora.com> Wed Jan 22 09:41:49 2020 +0100
tree e43c64db3102432dbeb529e2c4d1d08e74aaf928
parent e9262376f0acbe67a544d7e391992c9943ed6d8b

tdf#130112 lok: Do not apply zoom-factor twice...

... in ScOutputData::FillReferenceMarks().
mnPPT[XY] already has the factor aZoom[XY] in it. Refer
ScViewData::CalcPPT() to see how PPT[XY] are derived.

Change-Id: I3f9b5c01cb53514450fad5f7b2b6861b112effdb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87158
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
(cherry picked from commit 3f62c10548466119ec6b1a662ab339e5dbe0b05f)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87170
Tested-by: Jenkins
Reviewed-by: Dennis Francis <dennis.francis@collabora.com>

sc/source/ui/view/output.cxx

diff --git a/sc/source/ui/view/output.cxx b/sc/source/ui/view/output.cxx
index 605057e..35cd545 100644
--- a/sc/source/ui/view/output.cxx
+++ b/sc/source/ui/view/output.cxx
@@ -1958,10 +1958,11 @@

        if (bTop && bBottom && bLeft && bRight)
        {
            aResult = ReferenceMark( nMinX / mnPPTX * double( aZoomX ),
                                     nMinY / mnPPTY * double( aZoomY ),
                                     ( nMaxX - nMinX ) / mnPPTX * double( aZoomX ),
                                     ( nMaxY - nMinY ) / mnPPTY * double( aZoomY ),
            // mnPPT[XY] already has the factor aZoom[XY] in it.
            aResult = ReferenceMark( nMinX / mnPPTX,
                                     nMinY / mnPPTY,
                                     ( nMaxX - nMinX ) / mnPPTX,
                                     ( nMaxY - nMinY ) / mnPPTY,
                                     nTab,
                                     rColor );
        }