drawinglayer/source/primitive2d/textbreakuphelper.cxx |    2 +-
 drawinglayer/source/tools/wmfemfhelper.cxx            |    2 +-
 vcl/qa/cppunit/pdfexport/pdfexport2.cxx               |    4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 1f47a94170ddbba1acfd01a6c13c3585c6bd8573
Author:     Khaled Hosny <[email protected]>
AuthorDate: Tue Oct 21 13:11:52 2025 +0300
Commit:     Xisco Fauli <[email protected]>
CommitDate: Tue Oct 21 16:18:21 2025 +0200

    tdf#168884: Restore missing transparency in EMF files
    
    Revert unrelated changes from:
    
    commit 1bdf1e2d52a7fae3e1a58ddf15a7657df3d47004
    Author: Khaled Hosny <[email protected]>
    Date:   Thu Sep 25 16:33:09 2025 +0300
    
        tdf#168371: Disable ligatures in Impress/Draw with character spacing
    
    I added a new parameter to TextSimplePortionPrimitive2D constructor, and
    rTextFillColor that previously was using the default value had now to be
    explicitly set, but I should have used COL_TRANSPARENT as that is the
    default.
    
    Change-Id: Ie51e8f3f4c68a9539eb3f6cc77fb1e6cf44f673f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/192779
    Reviewed-by: Khaled Hosny <[email protected]>
    Tested-by: Jenkins
    (cherry picked from commit 1dce54d8fb070d3d4134317612c145a923497bb4)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/192789
    Reviewed-by: Adolfo Jayme Barrientos <[email protected]>

diff --git a/drawinglayer/source/primitive2d/textbreakuphelper.cxx 
b/drawinglayer/source/primitive2d/textbreakuphelper.cxx
index be3887d18363..b971fda62778 100644
--- a/drawinglayer/source/primitive2d/textbreakuphelper.cxx
+++ b/drawinglayer/source/primitive2d/textbreakuphelper.cxx
@@ -182,7 +182,7 @@ namespace drawinglayer::primitive2d
                         mrSource.getFontAttribute(),
                         mrSource.getLocale(),
                         mrSource.getFontColor(),
-                        mrSource.getTextFillColor(),
+                        COL_TRANSPARENT,
                         mrSource.getLetterSpacing()));
             }
         }
diff --git a/drawinglayer/source/tools/wmfemfhelper.cxx 
b/drawinglayer/source/tools/wmfemfhelper.cxx
index 00aff7114203..cbbeb2e93e4f 100644
--- a/drawinglayer/source/tools/wmfemfhelper.cxx
+++ b/drawinglayer/source/tools/wmfemfhelper.cxx
@@ -1157,7 +1157,7 @@ namespace wmfemfhelper
                     std::move(aFontAttribute),
                     std::move(aLocale),
                     aFontColor,
-                    aFillColor,
+                    COL_TRANSPARENT,
                     rFont.GetFixKerning());
             }
         }
diff --git a/vcl/qa/cppunit/pdfexport/pdfexport2.cxx 
b/vcl/qa/cppunit/pdfexport/pdfexport2.cxx
index fb9817fbff8b..fa07b33e549b 100644
--- a/vcl/qa/cppunit/pdfexport/pdfexport2.cxx
+++ b/vcl/qa/cppunit/pdfexport/pdfexport2.cxx
@@ -4152,9 +4152,9 @@ CPPUNIT_TEST_FIXTURE(PdfExportTest2, testTdf145873)
     int nPageObjectCount = pPdfPage->getObjectCount();
 
     // tdf#145873: Without the fix #1 in place, this test would have failed 
with
-    // - Expected: 107
+    // - Expected: 66
     // - Actual  : 3
-    CPPUNIT_ASSERT_EQUAL(107, nPageObjectCount);
+    CPPUNIT_ASSERT_EQUAL(66, nPageObjectCount);
 
     auto pObject = pPdfPage->getObject(4);
     CPPUNIT_ASSERT_MESSAGE("no object", pObject != nullptr);

Reply via email to