diff options
author | Attila Bakos <bakos.attilakaroly@nisz.hu> | 2020-09-17 11:32:44 +0200 |
---|---|---|
committer | Gabor Kelemen <kelemen.gabor2@nisz.hu> | 2020-10-15 10:06:28 +0200 |
commit | 3d536594cf3ecbd5b980aaddb5cf17084b1666d4 (patch) | |
tree | 0d0d4c35802dd936e44ac4a4baded97a685a0690 | |
parent | b16a19330820b3b7d11ca342ac0f26b13afd4f42 (diff) |
tdf#136613 sw: fix picture position in tables
in the case of option FollowTextFlow.
Regression from commit b6850bbe95418ecfde404be1696548f18d200c9b
(tdf#106153 sw compatibility: fix textboxes exceeding the page).
Note: now images have the same position than before the previous
commit, but this is still not perfect, because option FollowTextFlow
doesn't allow them to be placed outside the cell where they are
anchored to. The proper fix might be extending FollowTextFlow.
Change-Id: I877e4660cd4a2fd8f1e97c845755940459f1d014
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102910
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
(cherry picked from commit 7351a5028b18591306dfe18b34f6c751e565766e)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104330
Tested-by: Gabor Kelemen <kelemen.gabor2@nisz.hu>
Reviewed-by: Gabor Kelemen <kelemen.gabor2@nisz.hu>
-rw-r--r-- | sw/qa/extras/layout/data/tdf136613.docx | bin | 0 -> 25262 bytes | |||
-rw-r--r-- | sw/source/core/objectpositioning/anchoredobjectposition.cxx | 7 |
2 files changed, 6 insertions, 1 deletions
diff --git a/sw/qa/extras/layout/data/tdf136613.docx b/sw/qa/extras/layout/data/tdf136613.docx Binary files differnew file mode 100644 index 000000000000..f885564521b1 --- /dev/null +++ b/sw/qa/extras/layout/data/tdf136613.docx diff --git a/sw/source/core/objectpositioning/anchoredobjectposition.cxx b/sw/source/core/objectpositioning/anchoredobjectposition.cxx index 6935714e7c82..1e792ca09da3 100644 --- a/sw/source/core/objectpositioning/anchoredobjectposition.cxx +++ b/sw/source/core/objectpositioning/anchoredobjectposition.cxx @@ -409,8 +409,13 @@ SwTwips SwAnchoredObjectPosition::ImplAdjustVertRelPos( const SwTwips nTopOfAnch const bool bCheckBottom ) const { SwTwips nAdjustedRelPosY = nProposedRelPosY; - if (SwAnchoredObject::IsDraggingOffPageAllowed(FindFrameFormat(&mrDrawObj))) + // TODO: Replace the following condition with the correction + // of the implementation of option FollowTextFlow. + if ( SwAnchoredObject::IsDraggingOffPageAllowed(FindFrameFormat(&mrDrawObj)) && + !(GetAnchorFrame().IsInTab() && DoesObjFollowsTextFlow()) ) + { return nAdjustedRelPosY; + } const Size aObjSize(GetAnchoredObj().GetObjRect().SSize()); // determine the area of 'page' alignment frame, to which the vertical |