[Libreoffice-commits] core.git: sw/qa sw/source
Tibor Nagy (via logerrit)
logerrit at kemper.freedesktop.org
Mon Aug 24 15:31:49 UTC 2020
sw/qa/core/objectpositioning/data/bottom-margin-with-footer.docx |binary
sw/qa/core/objectpositioning/objectpositioning.cxx | 70 ++++++++++
sw/source/core/objectpositioning/anchoredobjectposition.cxx | 17 ++
3 files changed, 87 insertions(+)
New commits:
commit 6788133b3bdf02097d66a99047aa7bcba3a99a66
Author: Tibor Nagy <nagy.tibor2 at nisz.hu>
AuthorDate: Fri Aug 14 08:19:28 2020 +0200
Commit: László Németh <nemeth at numbertext.org>
CommitDate: Mon Aug 24 17:31:16 2020 +0200
tdf#135720 sw: fix PAGE_PRINT_AREA_BOTTOM alignment with footer
Top of the page margin area was the footer instead of the page
text area bottom, when the vertical position of the object was "top"
or "center" instead of "from top", resulting lower positions
after DOCX import.
Follow-up of commit d54264e34c1ccc0eaf4178ab165a843028500223
(tdf#133071 DOCX import: fix vertical alignment to bottom page margin)
Co-authored-by: Attila Szűcs (NISZ)
Change-Id: I4be46501fc4a3e5fb17330ea5f695a65014a7585
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100720
Tested-by: László Németh <nemeth at numbertext.org>
Reviewed-by: László Németh <nemeth at numbertext.org>
diff --git a/sw/qa/core/objectpositioning/data/bottom-margin-with-footer.docx b/sw/qa/core/objectpositioning/data/bottom-margin-with-footer.docx
new file mode 100644
index 000000000000..bb48ae4a7475
Binary files /dev/null and b/sw/qa/core/objectpositioning/data/bottom-margin-with-footer.docx differ
diff --git a/sw/qa/core/objectpositioning/objectpositioning.cxx b/sw/qa/core/objectpositioning/objectpositioning.cxx
index 58ea30b3ce53..21151f3b92de 100644
--- a/sw/qa/core/objectpositioning/objectpositioning.cxx
+++ b/sw/qa/core/objectpositioning/objectpositioning.cxx
@@ -141,6 +141,76 @@ CPPUNIT_TEST_FIXTURE(SwCoreObjectpositioningTest, testVertAlignBottomMargin)
// Verify that the distance between the bottom of body and top of third shape is around 0cm. (align=top)
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(0), nThirdShapeTop - nBodyBottom);
}
+
+CPPUNIT_TEST_FIXTURE(SwCoreObjectpositioningTest, testVertAlignBottomMarginWithFooter)
+{
+ // Load an empty document with footer.
+ load(DATA_DIRECTORY, "bottom-margin-with-footer.docx");
+ uno::Reference<css::lang::XMultiServiceFactory> xFactory(mxComponent, uno::UNO_QUERY);
+
+ // Insert three shapes and align it the bottom,center,top of page print area bottom.
+ // The height of page print area bottom is 2268 ~ 4cm.
+ // The size of shapes are 567 ~ 1cm
+ // Create first shape and align bottom of page print area bottom.
+ uno::Reference<drawing::XShape> xShapeBottom(
+ xFactory->createInstance("com.sun.star.drawing.RectangleShape"), uno::UNO_QUERY);
+ xShapeBottom->setSize(awt::Size(1000, 1000));
+ uno::Reference<beans::XPropertySet> xShapePropsBottom(xShapeBottom, uno::UNO_QUERY);
+ xShapePropsBottom->setPropertyValue("AnchorType",
+ uno::makeAny(text::TextContentAnchorType_AT_CHARACTER));
+ xShapePropsBottom->setPropertyValue("VertOrient", uno::makeAny(text::VertOrientation::BOTTOM));
+ xShapePropsBottom->setPropertyValue("VertOrientRelation",
+ uno::makeAny(text::RelOrientation::PAGE_PRINT_AREA_BOTTOM));
+ uno::Reference<drawing::XDrawPageSupplier> xDrawPageSupplierBottom(mxComponent, uno::UNO_QUERY);
+ xDrawPageSupplierBottom->getDrawPage()->add(xShapeBottom);
+
+ // Create second shape and align center of page print area bottom.
+ uno::Reference<drawing::XShape> xShapeCenter(
+ xFactory->createInstance("com.sun.star.drawing.RectangleShape"), uno::UNO_QUERY);
+ xShapeCenter->setSize(awt::Size(1000, 1000));
+ uno::Reference<beans::XPropertySet> xShapePropsCenter(xShapeCenter, uno::UNO_QUERY);
+ xShapePropsCenter->setPropertyValue("AnchorType",
+ uno::makeAny(text::TextContentAnchorType_AT_CHARACTER));
+ xShapePropsCenter->setPropertyValue("VertOrient", uno::makeAny(text::VertOrientation::CENTER));
+ xShapePropsCenter->setPropertyValue("VertOrientRelation",
+ uno::makeAny(text::RelOrientation::PAGE_PRINT_AREA_BOTTOM));
+ uno::Reference<drawing::XDrawPageSupplier> xDrawPageSupplierCenter(mxComponent, uno::UNO_QUERY);
+ xDrawPageSupplierCenter->getDrawPage()->add(xShapeCenter);
+
+ // Create third shape and align top of page print area bottom.
+ uno::Reference<drawing::XShape> xShapeTop(
+ xFactory->createInstance("com.sun.star.drawing.RectangleShape"), uno::UNO_QUERY);
+ xShapeTop->setSize(awt::Size(1000, 1000));
+ uno::Reference<beans::XPropertySet> xShapePropsTop(xShapeTop, uno::UNO_QUERY);
+ xShapePropsTop->setPropertyValue("AnchorType",
+ uno::makeAny(text::TextContentAnchorType_AT_CHARACTER));
+ xShapePropsTop->setPropertyValue("VertOrient", uno::makeAny(text::VertOrientation::TOP));
+ xShapePropsTop->setPropertyValue("VertOrientRelation",
+ uno::makeAny(text::RelOrientation::PAGE_PRINT_AREA_BOTTOM));
+ uno::Reference<drawing::XDrawPageSupplier> xDrawPageSupplierTop(mxComponent, uno::UNO_QUERY);
+ xDrawPageSupplierTop->getDrawPage()->add(xShapeTop);
+
+ xmlDocUniquePtr pXmlDoc = parseLayoutDump();
+ sal_Int32 nBodyBottom = getXPath(pXmlDoc, "//body/infos/bounds", "bottom").toInt32(); //14853
+ sal_Int32 nPageBottom = getXPath(pXmlDoc, "//page/infos/bounds", "bottom").toInt32(); //17121
+ sal_Int32 nFirstShapeBottom
+ = getXPath(pXmlDoc, "//SwAnchoredDrawObject[1]/bounds", "bottom").toInt32(); //17122
+ sal_Int32 nSecondShapeTop
+ = getXPath(pXmlDoc, "//SwAnchoredDrawObject[2]/bounds", "top").toInt32(); //15703
+ sal_Int32 nSecondShapeBottom
+ = getXPath(pXmlDoc, "//SwAnchoredDrawObject[2]/bounds", "bottom").toInt32(); //16272
+ sal_Int32 nThirdShapeTop
+ = getXPath(pXmlDoc, "//SwAnchoredDrawObject[3]/bounds", "top").toInt32(); //14853
+
+ // Verify that the distance between the bottom of page and bottom of first shape is around 0cm. (align=bottom)
+ CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), nFirstShapeBottom - nPageBottom);
+ // Verify that the distance between the bottom of page and bottom of second shape is around 1.5cm and
+ // verify that the distance between the bottom of body and top of second shape is around 1.5cm.(align=center)
+ CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(849), nPageBottom - nSecondShapeBottom);
+ CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(850), nSecondShapeTop - nBodyBottom);
+ // Verify that the distance between the bottom of body and top of third shape is around 0cm. (align=top)
+ CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(0), nThirdShapeTop - nBodyBottom);
+}
CPPUNIT_PLUGIN_IMPLEMENT();
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/core/objectpositioning/anchoredobjectposition.cxx b/sw/source/core/objectpositioning/anchoredobjectposition.cxx
index 236e4fafa6f0..d83eaffb106d 100644
--- a/sw/source/core/objectpositioning/anchoredobjectposition.cxx
+++ b/sw/source/core/objectpositioning/anchoredobjectposition.cxx
@@ -292,6 +292,23 @@ void SwAnchoredObjectPosition::GetVertAlignmentValues(
nOffset = aRectFnSet.YDiff(
aRectFnSet.GetPrtBottom(_rPageAlignLayFrame),
nVertOrientTop);
+
+ if (_rPageAlignLayFrame.IsPageFrame() && !aRectFnSet.IsVert())
+ {
+ const SwFrame* pPrtFrame =
+ static_cast<const SwPageFrame&>(_rPageAlignLayFrame).Lower();
+
+ while (pPrtFrame)
+ {
+ if (pPrtFrame->IsFooterFrame())
+ {
+ nHeight += pPrtFrame->getFrameArea().Height();
+ nOffset -= pPrtFrame->getFrameArea().Height();
+ }
+ pPrtFrame = pPrtFrame->GetNext();
+ }
+ }
+
}
break;
// #i22341# - vertical alignment at top of line
More information about the Libreoffice-commits
mailing list