[Libreoffice-commits] core.git: starmath/source starmath/visual-editor-todo

Ilmari Lauhakangas (via logerrit) logerrit at kemper.freedesktop.org
Fri Oct 23 14:20:54 UTC 2020


 starmath/source/cursor.cxx  |    1 -
 starmath/visual-editor-todo |    3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 508daea752c1d9c44474abcee43168fc12657541
Author:     Ilmari Lauhakangas <ilmari.lauhakangas at libreoffice.org>
AuthorDate: Fri Oct 23 15:23:14 2020 +0300
Commit:     Ilmari Lauhakangas <ilmari.lauhakangas at libreoffice.org>
CommitDate: Fri Oct 23 16:20:20 2020 +0200

    CreateTextFromNode is gone
    
    Change-Id: Ia5786a642a4eea57eef224ace384cb3f9da9fe75
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104728
    Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas at libreoffice.org>
    Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas at libreoffice.org>

diff --git a/starmath/source/cursor.cxx b/starmath/source/cursor.cxx
index 90bdf2207327..cdac53d4a234 100644
--- a/starmath/source/cursor.cxx
+++ b/starmath/source/cursor.cxx
@@ -1311,7 +1311,6 @@ void SmCursor::EndEdit(){
     //Update the edit engine and text of the document
     OUString formula;
     SmNodeToTextVisitor(mpTree, formula);
-    //mpTree->CreateTextFromNode(formula);
     mpDocShell->maText = formula;
     mpDocShell->GetEditEngine().QuickInsertText( formula, ESelection( 0, 0, EE_PARA_ALL, EE_TEXTPOS_ALL ) );
     mpDocShell->GetEditEngine().QuickFormatDoc();
diff --git a/starmath/visual-editor-todo b/starmath/visual-editor-todo
index 04dedae01699..65f2d7a7b6a0 100644
--- a/starmath/visual-editor-todo
+++ b/starmath/visual-editor-todo
@@ -15,8 +15,7 @@ Easy
 1. SmGraphicWindow::KeyInput relies on comparison of char, a better way must be available for CTRL+c
 2. Code style (missing spaces, linebreaks and a few renames)
 3. More documentation
-4. Remove the CreateTextFromNode methods and replace calls to it with NodeToTextVisitor
-5. Extend NodeToTextVisitor to update token offsets so SmNode::GetRow and SmNode::GetColumn will work.
+4. Extend NodeToTextVisitor to update token offsets so SmNode::GetRow and SmNode::GetColumn will work.
    (These methods can be used to enable synchronization of caret positions between visual and non-visual editor).
 
 Medium


More information about the Libreoffice-commits mailing list