Bug 305932

Summary:	Table's borders style changed on close
Product:	[Applications] calligrawords	Reporter:	Giuseppe Calà <gcala>
Component:	tables	Assignee:	Calligra Words Bugs <calligra-words-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	2.5.0
Target Milestone:	---
Platform:	Chakra
OS:	Linux
Latest Commit:	http://commits.kde.org/calligra/7fcb7a3423158397933fd0019e03c2b76d2d3f09	Version Fixed In:

Description Giuseppe Calà 2012-08-28 15:39:06 UTC

Table's border styles aren't preserved when closing and reopening a document. Here before/after samples:

http://i.imgur.com/LKEuy.png
http://i.imgur.com/GNNfJ.png

seems to happen with "unconventional" styles. The colour is preserved.

Reproducible: Always

Steps to Reproduce:
1. apply various style to table's borders
2. save & close
3. open
Actual Results:  
Some styles (double lines, broken lines) are replaced by plain lines

Expected Results:  
same look as beore closing the document

Comment 1 Camilla Boemann 2012-11-10 15:59:44 UTC

Git commit e6c1c31e56cc5ac189d56dad90d90e8b2c938773 by C. Boemann.
Committed on 09/11/2012 at 13:20.
Pushed by boemann into branch 'master'.

Use KoBorder to load the borders.

This gains us support of loading double borders. And since ms type borders
was ported to KoBorder in previous commit we retain loading and gain saving for that as well.

Drawback is that we now loose loading parentstyle borders. In the long run this is
correct as tablecellstyle should have inheritance like para and char styles, but for
now we have lost it. Good news is that I've never seen any odf file that uses this in
practice.

M  +5    -102  libs/kotext/styles/KoTableCellStyle.cpp

http://commits.kde.org/calligra/e6c1c31e56cc5ac189d56dad90d90e8b2c938773

Comment 2 Camilla Boemann 2012-11-28 11:54:35 UTC

Git commit 7fcb7a3423158397933fd0019e03c2b76d2d3f09 by C. Boemann.
Committed on 09/11/2012 at 13:20.
Pushed by boemann into branch 'calligra/2.6'.

Use KoBorder to load the borders.

This gains us support of loading double borders. And since ms type borders
was ported to KoBorder in previous commit we retain loading and gain saving for that as well.

Drawback is that we now loose loading parentstyle borders. In the long run this is
correct as tablecellstyle should have inheritance like para and char styles, but for
now we have lost it. Good news is that I've never seen any odf file that uses this in
practice.

M  +5    -102  libs/kotext/styles/KoTableCellStyle.cpp

http://commits.kde.org/calligra/7fcb7a3423158397933fd0019e03c2b76d2d3f09