diff options
author | Oliver Specht <oliver.specht@cib.de> | 2025-03-12 15:39:00 +0100 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2025-03-18 11:46:42 +0100 |
commit | a2a88f0073bdd5119745679daa61469e87002579 (patch) | |
tree | 735efd61f09f5a31d024a083ec2a1ea129339d7e /include/canvas/verifyinput.hxx | |
parent | 0c5953f1adb36ba3c5658050e6fb4b086839f210 (diff) |
Keeps reference to range in front of the current range to
apply page style to an already available page break in cases
where the section starts with a table.
Page number restart is also applied in the old and the new code.
Remove of dummy section paragraphs introduced with
b696600821d8aafb63b6a88016d299ef89478f56 is now only performed
if it is no RTF import.
This causes:
Test name: testHiddenParaBreaks::TestBody
equality assertion failed
- Expected: 3
- Actual : 4
- In <>, XPath '/root/page[2]/body/txt' number of nodes is incorrect
But the file is not imported correctly anyway and this is actually
fixing a bug, so adapt the test file.
rtfimport.cxx:910:Assertion
Test name: (anonymous namespace)::testContSectionPageBreak::TestBody
equality assertion failed
- Expected: THIRD
- Actual :
The problem here is that the extra paragraph that is inserted for \sect
if m_bIgnoreNextContSectBreak is now wrong as it was previously removed
but no longer is, so don't insert the extra paragraph any more.
But leave the extra paragraph in case it's a frame, otherwise we get:
rtfexport8.cxx:91:(anonymous namespace)::testTdf158586_0::TestBody
equality assertion failed
- Expected: 1
- Actual : 2
testTd112202 fails, but Oliver says that extra page 6 is an improvement.
Change-Id: Ie9a22f00e75c797762f52e6b865e32002051988c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182830
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'include/canvas/verifyinput.hxx')
0 files changed, 0 insertions, 0 deletions