<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100421/">http://git.reviewboard.kde.org/r/100421/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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!</pre>
<br />
<p>- Casper</p>
<br />
<p>On January 25th, 2011, 1:22 p.m., Lukáš Tvrdý wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra, Thorsten Zachmann and Casper Boemann.</div>
<div>By Lukáš Tvrdý.</div>
<p style="color: grey;"><i>Updated Jan. 25, 2011, 1:22 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/kotext/CMakeLists.txt <span style="color: grey">(115966e)</span></li>
<li>libs/kotext/KoTableOfContentsGeneratorInfo.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/kotext/KoTableOfContentsGeneratorInfo.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/kotext/KoText.h <span style="color: grey">(6bfb176)</span></li>
<li>libs/kotext/opendocument/KoTextLoader.h <span style="color: grey">(3aba119)</span></li>
<li>libs/kotext/opendocument/KoTextLoader.cpp <span style="color: grey">(e40d888)</span></li>
<li>libs/kotext/opendocument/KoTextWriter.cpp <span style="color: grey">(6567065)</span></li>
<li>plugins/textshape/ToCGenerator.h <span style="color: grey">(84747f1)</span></li>
<li>plugins/textshape/ToCGenerator.cpp <span style="color: grey">(3aa6a10)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100421/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>