A better fix for bug 3194 (see rev. 17027).

* BufferView::mouseSetCursor(): do not call setTargetX() here because this needs the mathed coordcache which is not ensured to be up to date. Because of this, cursor movement might need some adjustment.

* InsetMathNest::lfunMousePress(): revert commit 17027.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@17028 a592a061-630c-0410-9148-cb99ea01b6c8

diff --git a/src/BufferView.C b/src/BufferView.C
index a0da8a82d86ffa9b903ce3a424a6a275c6a3a930..1419335abc31a35cdf2fdb76069d3942488b082f 100644 (file)
--- a/src/BufferView.C
+++ b/src/BufferView.C
@@ -1326,8 +1326,6 @@ bool BufferView::mouseSetCursor(LCursor & cur)
 
        cursor_.setCursor(dit);
        cursor_.clearSelection();
-       // remember new position.
-       cursor_.setTargetX();
        finishUndo();
        return update;
 }

diff --git a/src/mathed/InsetMathNest.C b/src/mathed/InsetMathNest.C
index 1ba75523d07667770efb6c7b1f93059f30361c55..84883636161e650fb5e3f0bc8df5c288af326fd9 100644 (file)
--- a/src/mathed/InsetMathNest.C
+++ b/src/mathed/InsetMathNest.C
@@ -1160,6 +1160,7 @@ void InsetMathNest::lfunMousePress(LCursor & cur, FuncRequest & cmd)
                        asArray(theSelection().get(), ar);
 
                cur.insert(ar);
+               bv.mouseSetCursor(cur);
        }
 }