<div dir="ltr">Hi Weike,<div><br></div><div>If you are interested in contributing to LibreOffice, the best way to get your patch reviewed is via Gerrit.</div><div><br></div><div><a href="https://wiki.documentfoundation.org/Development/gerrit">https://wiki.documentfoundation.org/Development/gerrit</a><br></div><div><br></div><div>Regarding your patch, it ignores nLeftBorderDistance completely. You might need to further investigate why the value is wrong or not suitable for Writer to conclude that it should be ignored. Otherwise you're making it correct only for that document.</div><div><br></div><div>I also advice that you report an issue and upload the attachment, so it is can be refered when discussion.</div><div><a href="https://bugs.documentfoundation.org">https://bugs.documentfoundation.org</a><br></div><div><br></div><div><br></div><div><br></div><div> </div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-15 9:52 GMT+08:00  <span dir="ltr"><<a href="mailto:weike.liang@cs2c.com.cn" target="_blank">weike.liang@cs2c.com.cn</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p>Hi all,</p><p><br></p><p>Sorry for forgetting to attach the document. Please take a look at the position of the last table.</p><p><br></p><p>Best regards,</p><p>Weike Liang</p><p> </p><p> </p><div></div><p>​</p><p></p><p>-------- 转发邮件信息 --------</p><p></p><table border="0" cellpadding="0" cellspacing="0" width="100%" style="table-layout:fixed"><tbody><tr><th align="right" nowrap valign="baseline" width="60">主　题: </th><td>A question about table indent</td></tr><tr><th align="right" nowrap valign="baseline" width="60">日　期: </th><td>Mon, 14 Mar 2016 10:23:29 +0800</td></tr><tr><th align="right" nowrap valign="baseline" width="60">发件人: </th><td><<a href="mailto:weike.liang@cs2c.com.cn" target="_blank">weike.liang@cs2c.com.cn</a>></td></tr><tr><th align="right" nowrap valign="baseline" width="60">收件人: </th><td>libreoffice <<a href="mailto:libreoffice@lists.freedesktop.org" target="_blank">libreoffice@lists.freedesktop.org</a>></td></tr></tbody></table><div class="HOEnZb"><div class="h5"><p><br></p><p> Hi all,</p><p><br></p><p>My colleague from testing department reported a bug about the indent of the whole table. I looked into the source code and found the position where the problem may be. In writerilter/source/dmapper/DomainMapperTableHandler.cxx there exists an if-else statement that is responsible to calculate and process the table indent. But after i read the annotation i found the lines in if and else clauses were wrong. They both should change their positions with each other. I don't know if it is right. Please review the changes and give me some suggustions. Thanks.  </p><p><br></p><p>Best regards,</p><p>Weike Liang</p><p></p><p></p><div></div><p><br></p></div></div><br>_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Mark Hung<br></div></div></div>
</div>