Review Request: Table of content for words

[Casper Boemann]

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

Ship it!


If a heading has no text and no number, i Don't think it should show up in the ToC. People do ten to use such empty lines as linespace. Can you check with oo and msword and possibly fix.

Branch is good to be merged. You can do above correction in master or before merging. I don't care.

Great work guys!

- Casper


On Jan. 25, 2011, 1:22 p.m., Lukáš Tvrdý wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100421/
> -----------------------------------------------------------
> 
> (Updated Jan. 25, 2011, 1:22 p.m.)
> 
> 
> Review request for Calligra, Thorsten Zachmann and Casper Boemann.
> 
> 
> Summary
> -------
> 
> This patch implements the loading of the part of the ODF that says how should be the 
> table of content generated (table-of-content-source element with children).
> 
> Full loading is done, generation of the links is solved, navigation of the links works.
> 
> There are still some TODO's marked in the code. 
> I think the development can continue in master as the solid ground is done.
> 
> This bug was closed as invalid through the development of this feature:
> https://bugs.kde.org/show_bug.cgi?id=260542
> 
> Work done by Pavol Korinek and Lukas Tvrdy.
> 
> 
> Diffs
> -----
> 
>   libs/kotext/CMakeLists.txt 115966e 
>   libs/kotext/KoTableOfContentsGeneratorInfo.h PRE-CREATION 
>   libs/kotext/KoTableOfContentsGeneratorInfo.cpp PRE-CREATION 
>   libs/kotext/KoText.h 6bfb176 
>   libs/kotext/opendocument/KoTextLoader.h 3aba119 
>   libs/kotext/opendocument/KoTextLoader.cpp e40d888 
>   libs/kotext/opendocument/KoTextWriter.cpp 6567065 
>   plugins/textshape/ToCGenerator.h 84747f1 
>   plugins/textshape/ToCGenerator.cpp 3aa6a10 
> 
> Diff: http://git.reviewboard.kde.org/r/100421/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lukáš
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110128/944a0d07/attachment.htm>