<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/105030/">http://git.reviewboard.kde.org/r/105030/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 27th, 2012, 4:21 a.m., <b>Thorsten Zachmann</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There are still 2 classes in the code for parsing the data. I think it would be best if you add the new functionality of KPrFormulaParser to KPrValueParser and remove the KPrFormulaParser. If you think KPrFormulaParser is the better name please rename the class in a separate commit.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The class ValuesParser is no longer used, I just forget to remove it.</pre>
<br />
<p>- Paul</p>
<br />
<p>On May 27th, 2012, 3:04 p.m., Paul Mendez wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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.</div>
<div>By Paul Mendez.</div>
<p style="color: grey;"><i>Updated May 27, 2012, 3:04 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;">Add support for Anim:Formula tag in Stage. (That key is part of ODF animations specification).
Note: animations tested don't run as smooth as in LibreOffice because KeySplines tag is not implemented.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Test some animations of documents created in Libre Office (One test document is also uploaded)</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>stage/part/CMakeLists.txt <span style="color: grey">(3c7916ef7496af21e65d9a5441d5cb924829c347)</span></li>
<li>stage/part/animations/KPrAnimate.cpp <span style="color: grey">(dddd1fa401d596e7e23688f950428cd0ea76b639)</span></li>
<li>stage/part/animations/strategy/KPrFormulaParser.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>stage/part/animations/strategy/KPrFormulaParser.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>stage/part/animations/strategy/KPrSmilValues.h <span style="color: grey">(163d78b830a151ce150192000890a395f9e273dd)</span></li>
<li>stage/part/animations/strategy/KPrSmilValues.cpp <span style="color: grey">(3faafc4eb1c8783224f9f32c38106407cc219096)</span></li>
<li>stage/part/animations/strategy/KPrValueParser.h <span style="color: grey">(8f3c6ebcdf7ae9f5d938d97d518f21977b572940)</span></li>
<li>stage/part/animations/strategy/KPrValueParser.cpp <span style="color: grey">(ca7fac767142e64b9711ccf835efcb9a4c242e36)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105030/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>