[Libreoffice] [REVIEW][3.5.0][3-5][3-4] fdo#38595 Calc cell borders lost in ODF import
Michael Stahl
mstahl at redhat.com
Thu Jan 26 09:21:16 PST 2012
On 26/01/12 14:20, Eike Rathke wrote:
> Hi,
>
> please review and cherry-pick to 3-5, 3.5.0 and 3-4
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=34315e7ec4062f9521cd19951b5f7f6ad9ce0d2e
>
> Resolves https://bugs.freedesktop.org/show_bug.cgi?id=38595
ooh, Calc also has this kind of problem :)
your fix is an improvement, but i have doubts about taking the width
from the fo:border at all if there is a style:border-line-width:
the idea is that for double borders, the 3 parts of
style:border-line-width should add up to the total width of the border;
so if the fo:border contains a value different from the sum of the 3
parts then there is probably a problem.. i'd call such a document invalid.
but i suppose it would be legal to omit the width from fo:border for
double borders? what does this fix do then? initialize the width to zero?
i've fixed this differently in my patch for writer, which Cedric has
promised to review any minute now:
https://bugs.freedesktop.org/attachment.cgi?id=56151
More information about the LibreOffice
mailing list