Review Request 109994: New implementation for handling formulas
Thorsten Zachmann
t.zachmann at zagge.de
Sun Apr 14 07:10:22 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109994/#review31009
-----------------------------------------------------------
libs/odf/KoXmlReader.cpp
<http://git.reviewboard.kde.org/r/109994/#comment23026>
I'm not so sure about this change. Maybe you can send a direct mail to Ariya (ariya at kde.org) and ask about it. It usually takes some days before he answers.
Should the problem not be avoided when opening the document without namespace processing? If we would add a flag to KoOdfReadStore to switch off namespace processing this could be fixed.
plugins/staging/CMakeLists.txt
<http://git.reviewboard.kde.org/r/109994/#comment23027>
This is unrelated to you work and should go.
plugins/staging/formulashape-2g/CMakeLists.txt
<http://git.reviewboard.kde.org/r/109994/#comment23029>
Can you please remove all the additional Space at the end of line as shown by reviewboard?
plugins/staging/formulashape-2g/ChangeFormulaCommand.cpp
<http://git.reviewboard.kde.org/r/109994/#comment23028>
I guess you meant m_isLastCommandRedo = false;?
plugins/staging/formulashape-2g/FormulaShape.cpp
<http://git.reviewboard.kde.org/r/109994/#comment23030>
is there a reason m_document and m_cursor are not deleted here?
- Thorsten Zachmann
On April 13, 2013, 3:03 p.m., Gopalakrishna Bhat wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109994/
> -----------------------------------------------------------
>
> (Updated April 13, 2013, 3:03 p.m.)
>
>
> Review request for Calligra.
>
>
> Description
> -------
>
> 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 :)
>
>
> Diffs
> -----
>
> libs/odf/KoXmlReader.cpp b5fe9f8
> plugins/staging/formulashape-2g/Messages.sh PRE-CREATION
> plugins/staging/formulashape-2g/FormulaToolFactory.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaToolFactory.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaTool.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaTool.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShapePlugin.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShapePlugin.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShapeFactory.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShapeFactory.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShape.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaShape.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaPart.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaPart.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaDocument.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaDocument.h PRE-CREATION
> plugins/staging/formulashape-2g/FormulaCursor.cpp PRE-CREATION
> plugins/staging/formulashape-2g/FormulaCursor.h PRE-CREATION
> plugins/staging/formulashape-2g/ChangeFormulaCommand.cpp PRE-CREATION
> plugins/staging/formulashape-2g/ChangeFormulaCommand.h PRE-CREATION
> plugins/staging/formulashape-2g/CMakeLists.txt PRE-CREATION
> plugins/staging/formulashape-2g/AlterFormulaCommand.cpp PRE-CREATION
> plugins/staging/formulashape-2g/AlterFormulaCommand.h PRE-CREATION
> plugins/staging/CMakeLists.txt 754362f
> plugins/staging/formulashape-2g/formula_export.h PRE-CREATION
> plugins/staging/formulashape-2g/formulashape2g.desktop PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/LGPL_EXCEPTION.txt PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/LICENSE.GPL3 PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/LICENSE.LGPL PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/QtMmlDocument PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/QtMmlWidget PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/README.TXT PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/qtmmlwidget.h PRE-CREATION
> plugins/staging/formulashape-2g/qtmmlwidget/qtmmlwidget.cpp PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/109994/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> Snapshot
> http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/formulaShape.png
>
>
> Thanks,
>
> Gopalakrishna Bhat
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130414/67386340/attachment.htm>
More information about the calligra-devel
mailing list