diff options
| author | Xisco Fauli <xiscofauli@libreoffice.org> | 2023-09-06 11:35:26 +0200 |
|---|---|---|
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2023-09-06 14:13:41 +0200 |
| commit | bc25df351f946c799112533af1cf9b1dcb2f6079 (patch) | |
| tree | 76d9653d6ea23a83c689efe069942e6e6a757e11 /svgio | |
| parent | d29c1a90ae77dde7c87c51f21e859fa254f23e01 (diff) | |
tdf#157113: Add gap even if baseline is different
This was a workaround for https://bz.apache.org/ooo/show_bug.cgi?id=122524
but it's no longer needed since 5079e7937ef471a44dcf119dc6ae0a334d9c6adc
"tdf#156251: Add gap between text elements when needed"
Change-Id: I83af59a515d56af6b0e3cf3e351d3df52510a17d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156613
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
Diffstat (limited to 'svgio')
| -rw-r--r-- | svgio/qa/cppunit/SvgImportTest.cxx | 2 | ||||
| -rw-r--r-- | svgio/source/svgreader/svgcharacternode.cxx | 11 |
2 files changed, 1 insertions, 12 deletions
diff --git a/svgio/qa/cppunit/SvgImportTest.cxx b/svgio/qa/cppunit/SvgImportTest.cxx index c0792504cfba..b337418dbdec 100644 --- a/svgio/qa/cppunit/SvgImportTest.cxx +++ b/svgio/qa/cppunit/SvgImportTest.cxx @@ -1842,7 +1842,7 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf156837) // - Actual : 103 assertXPath(pDocument, "/primitive2D/transform/textsimpleportion[2]", "y", "94"); assertXPath(pDocument, "/primitive2D/transform/textsimpleportion[2]", "height", "10"); - assertXPath(pDocument, "/primitive2D/transform/textsimpleportion[2]", "text", "3"); + assertXPath(pDocument, "/primitive2D/transform/textsimpleportion[2]", "text", " 3"); } CPPUNIT_TEST_FIXTURE(Test, testTdf156271) diff --git a/svgio/source/svgreader/svgcharacternode.cxx b/svgio/source/svgreader/svgcharacternode.cxx index 0199a1ff9105..222c2739a689 100644 --- a/svgio/source/svgreader/svgcharacternode.cxx +++ b/svgio/source/svgreader/svgcharacternode.cxx @@ -514,17 +514,6 @@ namespace svgio::svgreader if(pPreviousCharacterNode->mpParentLine != mpParentLine) bAddGap = false; - // With this option a baseline shift between two char parts ('words') - // will not add a space 'gap' to the end of the (non-last) word. This - // seems to be the standard behaviour, see last bugdoc attached #122524# - const SvgStyleAttributes* pStyleLast = pPreviousCharacterNode->getSvgStyleAttributes(); - const SvgStyleAttributes* pStyleCurrent = getSvgStyleAttributes(); - - if(pStyleLast && pStyleCurrent && pStyleLast->getBaselineShift() != pStyleCurrent->getBaselineShift()) - { - bAddGap = false; - } - // add in-between whitespace (single space) to last // known character node if(bAddGap) |