tdf#140342 sw layout: remove explicit gutter handling when positioning borders

Word has two modes the specify border positions: the distance can be
measured from the edge of the page or from text. Similar to how
documents that have page borders but no gutter, page page border +
gutter documents should be always measured "from text" by the layout.

"From page" should be a concern for Word filters.

"From page" was already working -- fix "from text" by changing the
layout to do "from text" and then extending DOCX import/export to handle
gutter while handling the "from page" page borders.

This also requires allowing nominally negative top margins, because we
want to have the gutter unchanged, but the border might want to be on
the gutter area, which is only possible with a negative top margin.

Change-Id: I7f2c9943357359e76cb554cb2a65b93a492e694b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111735
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

2 files changed, 9 insertions, 27 deletions

diff --git a/sw/source/core/layout/paintfrm.cxx b/sw/source/core/layout/paintfrm.cxx
index ef69c5ff81ba..b3c940b87cea 100644
--- a/sw/source/core/layout/paintfrm.cxx
+++ b/sw/source/core/layout/paintfrm.cxx
@@ -1289,33 +1289,6 @@ static void lcl_CalcBorderRect( SwRect &rRect, const SwFrame *pFrame,
 
         SwRectFn fnRect = pFrame->IsVertical() ? ( pFrame->IsVertLR() ? (pFrame->IsVertLRBT() ? fnRectVertL2RB2T : fnRectVertL2R) : fnRectVert ) : fnRectHori;
 
-        if (pFrame->IsPageFrame() && rAttrs.GetLRSpace())
-        {
-            tools::Long nGutterMargin = rAttrs.GetLRSpace()->GetGutterMargin();
-            tools::Long nRightGutterMargin = rAttrs.GetLRSpace()->GetRightGutterMargin();
-            const auto pPageFrame = static_cast<const SwPageFrame*>(pFrame);
-            bool bGutterAtTop = pPageFrame->GetFormat()->getIDocumentSettingAccess().get(
-                DocumentSettingId::GUTTER_AT_TOP);
-            if (bGutterAtTop)
-            {
-                // Paint the top border based on the top margin, ignoring the gutter margin.
-                (rRect.*fnRect->fnSubTop)(nGutterMargin);
-            }
-            else
-            {
-                if (nGutterMargin)
-                {
-                    // Paint the left border based on the left margin, ignoring the gutter margin.
-                    (rRect.*fnRect->fnSubLeft)(nGutterMargin);
-                }
-                if (nRightGutterMargin)
-                {
-                    // Paint the right border based on the right margin, ignoring the gutter margin.
-                    (rRect.*fnRect->fnAddRight)(nRightGutterMargin);
-                }
-            }
-        }
-
         const SvxBoxItem &rBox = rAttrs.GetBox();
         const bool bTop = 0 != (pFrame->*fnRect->fnGetTopMargin)();
         if ( bTop )
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx b/sw/source/filter/ww8/docxattributeoutput.cxx
index b5a793ec196f..67d6aa748df4 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -6743,6 +6743,15 @@ void DocxAttributeOutput::SectionPageBorders( const SwFrameFormat* pFormat, cons
     if (aGlue.HasFooter())
         aMargins.nBottom = aGlue.dyaHdrBottom;
 
+    if (pFormat->GetDoc()->getIDocumentSettingAccess().get(DocumentSettingId::GUTTER_AT_TOP))
+    {
+        aMargins.nTop += pFormat->GetLRSpace().GetGutterMargin();
+    }
+    else
+    {
+        aMargins.nLeft += pFormat->GetLRSpace().GetGutterMargin();
+    }
+
     aOutputBorderOptions.pDistances = std::make_shared<editeng::WordBorderDistances>();
     editeng::BorderDistancesToWord(rBox, aMargins, *aOutputBorderOptions.pDistances);