Thanks ! It exciting to join LO developers! I will confirmed the bug tdf#96412 <https://bugs.documentfoundation.org/show_bug.cgi?id=96412> and fix it wich report by myself.
https://bugs.documentfoundation.org/show_bug.cgi?id=96412 I think I've get it. I will push it to the gerrit after test on my PC. ------------------------------ [email protected] *From:* jan iversen <[email protected]> *Date:* 2015-12-15 20:56 *To:* [email protected] *CC:* libreoffice <[email protected]> *Subject:* Re: fix paint table border in DOUBLE_THIN style Job well done !! your patch is now on master. What will you work on next ? rgds jan i Sent from my iPad, please excuse any misspellings On 14 Dec 2015, at 16:26, "[email protected]" <[email protected]> wrote: Hi, All! I had push the commit to the gerrit.: https://gerrit.libreoffice.org/#/c/20701/ But it build failed in 'CppunitTest/libtest_writerperfect_stream.so': http://ci.libreoffice.org/job/lo_gerrit_master/9226/ It work well in my machine. Is anyone have a good idear to fix this bug ? Thanks! ------------------------------ [email protected] *From:* [email protected] *Date:* 2015-12-13 20:26 *To:* libreoffice <[email protected]> *Subject:* fix paint table border in DOUBLE_THIN style Hi, I find a bug in the master branch and fixed it. But i am in trouble when i push the patch to the gerrit. It the waring fllow: The authenticity of host '[gerrit.libreoffice.org ]:29418 ([89.238.68.148]:29418)' can't be established. So, i show the screenshot to show the bug and patch fllow: screenshot: http://7xjcd4.com1.z0.glb.clouddn.com/bog_lo_bug_cell_double_thin_1.png patch: >From f7d4b14cccb20ea39a90fb68986fb5578c6fd9a2 Mon Sep 17 00:00:00 2001 From: aqcoder <[email protected]> Date: Sun, 13 Dec 2015 19:11:28 +0800 Subject: [PATCH] fix paint table border in DOUBLE_THIN style it always paint partial table border which have line style of DOUBLE_THIN. Becouse the partial of the border line is out of the invalid window rect. so, we extend it. Change-Id: I9d95ca71a96cfa869c68d9d2200f29c7cd447eca --- sw/source/core/layout/paintfrm.cxx | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/sw/source/core/layout/paintfrm.cxx b/sw/source/core/layout/paintfrm.cxx index 83a8638..9746df7 100644 --- a/sw/source/core/layout/paintfrm.cxx +++ b/sw/source/core/layout/paintfrm.cxx @@ -2742,6 +2742,24 @@ void SwTabFramePainter::PaintLines(OutputDevice& rDev, const SwRect& rRect) cons aPaintEnd.Y() += static_cast<long>(offsetEnd + 0.5); } + if( rEntryStyle.Type() == table::BorderLineStyle::DOUBLE_THIN ) + { + long aPixel = rDev.PixelToLogic( Point(1, 1) ).getX(); + SwRect aPaintEx( aPaintStart, aPaintEnd ); + if( bHori ) + { + aPaintEx.Pos().Y() -= aPixel; + aPaintEx.SSize().Height() += aPixel * 2; + } + else + { + aPaintEx.Pos().X() -= aPixel; + aPaintEx.SSize().Width() += aPixel * 2; + } + + gProp.pSGlobalShell->InvalidateWindows( aPaintEx ); + } + if (bHori) { mrTabFrame.ProcessPrimitives( svx::frame::CreateBorderPrimitives( -- 2.5.0 _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
_______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
