diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2024-05-30 08:10:13 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2024-05-30 13:30:40 +0200 |
commit | 304cc248f15d795bc9fe8b751b0d98841b8ff5d1 (patch) | |
tree | a64a31571f8db2e1c5e0acb6ac0bb4007dca917d /include/svtools/htmlcfg.hxx | |
parent | cool#9174 - Enable .uno:ChangeBezier (diff) | |
download | core-master.tar.gz core-master.zip |
tdf#161318 sw clearing break: ignore wrap-through anchored objects
HEADmaster
Import the bugdoc: we expect just a single page for the 2 paragraphs,
but the second paragraph goes to a 2nd page.
What seems to happen since commit
f86d1482bef285f90079b5130e410646db96cf58 (sw clearing breaks: add DOCX
import, 2022-03-08) is that on one hand, SwTextFly::ForEach() ignores
wrap-through shapes, but at the same time SwTextFly::GetMaxBottom() did
not, and this lead to a loop: the first paragraph kept growing till the
shape was shifted to page 2, but then the same first paragraph was
reduced to just 2 lines, goto 1.
Fix the problem by extending SwTextFly::GetMaxBottom() to also ignore
wrap-though shapes: this is the intention, just clearing breaks were not
tested with wrap-though wrap mode before.
This is a reduced bugdoc, the original one even produced warnings like:
warn:legacy.osl:12034:12034:sw/source/core/layout/flowfrm.cxx:2667: <SwFlowFrame::MoveBwd(..)> - layout loop control for layout action <Move Backward> applied!
without the fix, and these are now gone.
Change-Id: Iaf9849dbf8e1a8e5d625d3c19b99636247804cdd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/168239
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'include/svtools/htmlcfg.hxx')
0 files changed, 0 insertions, 0 deletions