<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/109994/">http://git.reviewboard.kde.org/r/109994/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi Gopalakrishna.
Sorry for taking so long to review this, somehow I missed that there was a new version. Anyway, I have built the branch now and tested it.
There is something strange going on here. I created a couple of very simple formulas in LibreOffice and then loaded that file. I don't see the rendered formulas but I do see the LO math language, so probably you are loading the wrong part of the formula into the widget. Did you test the new shape only with formulas that were created in Calligra? LO creates some annotation in MathML with the formulas themselves, which Calligra doesn't do, so it could be something like that.</pre>
<br />
<p>- Inge</p>
<br />
<p>On April 19th, 2013, 2:25 p.m. UTC, Gopalakrishna Bhat wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By Gopalakrishna Bhat.</div>
<p style="color: grey;"><i>Updated April 19, 2013, 2:25 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 new implementation for formulashape uses and improves on qtmmlwidgets that was part of Qt Solutions. The code can be found in plugins-formulashape2g-gopalK branch.
Currently implemented features are
1) Loading of formulas in document.
2) Insert of MathML formulas generated from some other tools(viz FireMath plugin for Firefox)
3) Editing of existing formulas.
Features that were there in old version of formulashape but missing now
1) Insert of Elements, Symbols (in a very much unusable state)
2) Edit formulas with LaTeX syntax (Will be done in coming days)
I will remove the extra whitespace that exists in the uploaded diff before merging :)</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>plugins/staging/CMakeLists.txt <span style="color: grey">(754362f)</span></li>
<li>plugins/staging/formulashape-2g/AlterFormulaCommand.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/AlterFormulaCommand.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/ChangeFormulaCommand.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/ChangeFormulaCommand.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaCursor.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaCursor.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaDocument.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaDocument.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaPart.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaPart.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShape.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShape.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShapeFactory.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShapeFactory.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShapePlugin.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaShapePlugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaTool.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaTool.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaToolFactory.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/FormulaToolFactory.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/Messages.sh <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/formula_export.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/formulashape2g.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/LGPL_EXCEPTION.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/LICENSE.GPL3 <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/LICENSE.LGPL <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/QtMmlDocument <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/QtMmlWidget <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/README.TXT <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/qtmmlwidget.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/staging/formulashape-2g/qtmmlwidget/qtmmlwidget.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109994/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/formulaShape.png">Snapshot</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>