[PATCH libreoffice-4-0] fdo#60462 - Small errors in the layout margins of the text b...

Joren De Cuyper (via Code Review) gerrit at gerrit.libreoffice.org
Sun Mar 24 12:47:14 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2972

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/72/2972/1

fdo#60462 - Small errors in the layout margins of the text boxes

As you can see on this screenshot
https://bugs.freedesktop.org/attachment.cgi?id=74399 there was a margin
problem. The X-position of the text box is at 0.05, but the
width was only 0.88 so we had 0.02 _extra_ on the right side. This is the
width of that little margin error. Fixing it by setting Layoutsize width
to 0.90, so we have on both side a nice 0.05 margin.

Change-Id: Id712ce3695a760c8f955413c9392c008733d7fb9
---
M sd/source/core/sdpage.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sd/source/core/sdpage.cxx b/sd/source/core/sdpage.cxx
index 5c4337f..e943e89 100644
--- a/sd/source/core/sdpage.cxx
+++ b/sd/source/core/sdpage.cxx
@@ -1052,7 +1052,7 @@
         {
             aLayoutPos.X() += long( aLayoutSize.Width() * 0.05 );
             aLayoutPos.Y() += long( aLayoutSize.Height() * 0.234 );
-            aLayoutSize.Width() = long( aLayoutSize.Width() * 0.88 );
+            aLayoutSize.Width() = long( aLayoutSize.Width() * 0.9 );
             aLayoutSize.Height() = long( aLayoutSize.Height() * 0.58 );
             aLayoutRect.SetPos(aLayoutPos);
             aLayoutRect.SetSize(aLayoutSize);

-- 
To view, visit https://gerrit.libreoffice.org/2972
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id712ce3695a760c8f955413c9392c008733d7fb9
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Joren De Cuyper <joren.libreoffice at telenet.be>



More information about the LibreOffice mailing list