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. ------------------------------ flw.aquar...@gmail.com *From:* jan iversen <j...@documentfoundation.org> *Date:* 2015-12-15 20:56 *To:* flw.aquar...@gmail.com *CC:* libreoffice <libreoffice@lists.freedesktop.org> *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, "flw.aquar...@gmail.com" <flw.aquar...@gmail.com> 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! ------------------------------ flw.aquar...@gmail.com *From:* flw.aquar...@gmail.com *Date:* 2015-12-13 20:26 *To:* libreoffice <libreoffice@lists.freedesktop.org> *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 <flw.aquar...@gmail.com> 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 LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice