Review Request: Refactor kotext/KoTableCellStyle
Pierre Ducroquet
pinaraf at gmail.com
Wed Aug 3 21:50:00 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102200/
-----------------------------------------------------------
(Updated Aug. 3, 2011, 8:50 p.m.)
Review request for Calligra.
Changes
-------
Fix whitespaces
Summary
-------
First refactoring of KoText table borders :
- get rid of the rendering code that was sitting in kotext/styles and move it to its own class in textlayout
- simplify KoTableCellStyle by kicking KoTableBorderStyle and removing the edges/borders private members
Next step : get rid of most of these properties and replace them with a KoBorder property.
Diffs (updated)
-----
filters/libmsooxml/MsooXmlDrawingTableStyleReader.cpp 849d4bad4d5b6e9f7a658d1f9def3092486c0b27
filters/words/docx/DocxXmlDocumentReader.cpp 98ddcf56cfbc08c6d109a9f2b4967695f6022cf9
libs/kotext/CMakeLists.txt 46c194fc27d3fd6437675afd90e96833e75df7f0
libs/kotext/KoTextBlockBorderData.cpp c829b5cd857f004d60002a09b2ba38b4b60bd065
libs/kotext/KoTextDebug.cpp 3d65b666efb1dfee24b9054e5c6346d454bd31df
libs/kotext/KoTextEditor.cpp 70c421d6011e7871a17acca1b2a91f80cf553c0f
libs/kotext/opendocument/tests/TestLoading.cpp c36deda2816a2709e965208adca41d41a7e17672
libs/kotext/styles/KoTableBorderStyle.h 2bb46dd9c91398ef29928de97a027d3e09a73b2a
libs/kotext/styles/KoTableBorderStyle.cpp ab88d8d2d555996d38d900a9ebb6bada668378f7
libs/kotext/styles/KoTableBorderStyle_p.h ca9d32d2466fbb808791a3d23618b2e3bb778a34
libs/kotext/styles/KoTableCellStyle.h dfb6967b846b21e6608e74a55f06441265df900b
libs/kotext/styles/KoTableCellStyle.cpp f68cd03d93f048dbfcded91d53e51c2737c4b7a9
libs/kotext/styles/KoTableCellStyle_p.h 3935c8190bc4f0c8c40f01886d9a7279fc22b689
libs/kotext/styles/tests/TestTableCellStyle.cpp e752e239db388f9144d13f575da0b1c52abc1689
libs/odf/KoBorder.h c7b9b5d97772569026dfc6bf3723ee93f7331f8a
libs/odf/KoBorder.cpp 4772305867b260acd76dab549c9de5ebeaaceb6a
libs/textlayout/CMakeLists.txt a364ac50240e8bd6e93d9aca73c43ae426d1bfd1
libs/textlayout/KoTextLayoutCellHelper.h PRE-CREATION
libs/textlayout/KoTextLayoutCellHelper.cpp PRE-CREATION
libs/textlayout/KoTextLayoutTableArea.cpp 28aebe3a949fd089619ab686a984ccc9333160ee
words/part/KWCanvasBase.cpp 9589f08dd172b097771369929f777e35b4d7f7dc
Diff: http://git.reviewboard.kde.org/r/102200/diff
Testing
-------
- Compilation
- Running unit tests
- Rendering of sample documents
Thanks,
Pierre
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110803/f846ab8c/attachment.htm>
More information about the calligra-devel
mailing list