sw/source/core/text/frmform.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 88661e5ca78638aa40f051b72844814a90e20dc3
Author: Miklos Vajna <vmik...@collabora.com>
AuthorDate: Thu Aug 29 10:13:51 2024 +0200
Commit: Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Thu Aug 29 14:28:53 2024 +0200
sw: call HasFullPageFly() later in SwTextFrame::FormatAdjust()
Suggested at
<https://gerrit.libreoffice.org/c/core/+/172562/2#message-01333d4017922b8d2c834519ea7d94517110112a>,
the other conditions are trivial to check, so best if we do the
expensive condition only at the end.
Change-Id: I54e4914095e2e06cf162e340754ba7859f0903d6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172586
Reviewed-by: Miklos Vajna <vmik...@collabora.com>
Tested-by: Jenkins
(cherry picked from commit 710fa4e68cc134d9970a6d0c034a84c3419be14a)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172575
Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
diff --git a/sw/source/core/text/frmform.cxx b/sw/source/core/text/frmform.cxx
index 82a0f8c7dc14..72856363a8bc 100644
--- a/sw/source/core/text/frmform.cxx
+++ b/sw/source/core/text/frmform.cxx
@@ -1266,7 +1266,7 @@ void SwTextFrame::FormatAdjust( SwTextFormatter &rLine,
}
}
- if (nNew && HasFullPageFly(this) && nEnd == TextFrameIndex(0) &&
!bEmptyWithSplitFly)
+ if (nNew && nEnd == TextFrameIndex(0) && !bEmptyWithSplitFly &&
HasFullPageFly(this))
{
// We intended to split at start, due to an anchored object which
would use all space on the
// current page. It makes no sense to split & move all text of the
frame forward: the
