Review Request: Continue refactoring of Kotext/styles/KoTableBorderStyle and KoBorder

C. Boemann cbr at boemann.dk
Wed Sep 28 11:22:11 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102724/#review6884
-----------------------------------------------------------


As I already said last night it looks ok in general, however when running cstester it revealed that outer borders of tables seem to have change color

and there is also a document where the table seem to have shifted position for no good reason (that maybe something else though).

for ref these are reminders of documents:
www.une.edu
www.alpineskihire


- C. Boemann


On Sept. 27, 2011, 11:49 p.m., Pierre Ducroquet wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102724/
> -----------------------------------------------------------
> 
> (Updated Sept. 27, 2011, 11:49 p.m.)
> 
> 
> Review request for Calligra and C. Boemann.
> 
> 
> Description
> -------
> 
> This patch removes almost everything that was left in KoTableBorderStyle and uses only KoBorder instead.
> 
> 
> Diffs
> -----
> 
>   filters/libmsooxml/MsooXmlDrawingTableStyleReader.cpp 86f07b1 
>   filters/libmsooxml/MsooXmlTableStyle.cpp 5f1de19 
>   filters/words/docx/DocxXmlDocumentReader.cpp 18f43fb 
>   filters/words/docx/DocxXmlStylesReader.cpp c859eca 
>   libs/kotext/KoTextEditor.cpp 9ff740c 
>   libs/kotext/styles/KoTableBorderStyle.h 6c24a75 
>   libs/kotext/styles/KoTableCellStyle.h 3d3379b 
>   libs/kotext/styles/KoTableCellStyle.cpp 1b6876c 
>   libs/odf/KoBorder.h 83cdac8 
>   libs/odf/KoBorder.cpp 1161a0a 
>   libs/textlayout/KoTextLayoutCellHelper.cpp 7d0dfd7 
>   words/part/KWCanvasBase.cpp fb7fc2c 
> 
> Diff: http://git.reviewboard.kde.org/r/102724/diff/diff
> 
> 
> Testing
> -------
> 
> Compiling, running kotext unit tests.
> 
> 
> Thanks,
> 
> Pierre Ducroquet
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110928/7ac1abd4/attachment.htm>


More information about the calligra-devel mailing list