do not create invalid .lyx files when importing \i{}, \j{}, \l{} or \L{}.

[lyx.git] / src / text3.C

diff --git a/src/text3.C b/src/text3.C
index f74cc67f178bdd295ae4a55ba5a02bab65a7e40e..f7d9a8806df63939c6974acf49b38d85feff05fc 100644 (file)
--- a/src/text3.C
+++ b/src/text3.C
@@ -207,12 +207,18 @@ void LyXText::cursorPrevious(LCursor & cur)
 
        // FIXME: there would maybe a need for this 'updated' boolean in the future...
        bool updated = setCursorFromCoordinates(cur, x, 0);
-       updated |= cursorUp(cur);
+       if (cur.inMathed())
+               updated |= cur.up();
+       else
+               updated |= cursorUp(cur);
 
        if (cpar == cur.pit() && cpos == cur.pos()) {
                // we have a row which is taller than the workarea. The
                // simplest solution is to move to the previous row instead.
-               updated |= cursorUp(cur);
+               if (cur.inMathed())
+                       updated |= cur.up();
+               else
+                       updated |= cursorUp(cur);
        }
 
        finishUndo();
@@ -228,12 +234,18 @@ void LyXText::cursorNext(LCursor & cur)
        int x = cur.x_target();
        // FIXME: there would maybe a need for this 'updated' boolean in the future...
        bool updated = setCursorFromCoordinates(cur, x, cur.bv().workHeight() - 1);
-       updated |= cursorDown(cur);
+       if (cur.inMathed())
+               updated |= cur.down();
+       else
+               updated |= cursorDown(cur);
 
        if (cpar == cur.pit() && cpos == cur.pos()) {
                // we have a row which is taller than the workarea. The
                // simplest solution is to move to the next row instead.
-               updated |= cursorDown(cur);
+               if (cur.inMathed())
+                       updated |= cur.down();
+               else
+                       updated |= cursorDown(cur);
        }
 
        finishUndo();
@@ -979,7 +991,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                        paste_internally = true;
                }
 
-               bv->mouseSetCursor(cur);
+               // we have to update after dePM triggered
+               bool update = bv->mouseSetCursor(cur);
 
                // Insert primary selection with middle mouse
                // if there is a local selection in the current buffer,
@@ -991,7 +1004,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                                lyx::dispatch(FuncRequest(LFUN_PRIMARY_SELECTION_PASTE, "paragraph"));
                }
 
-               if (cmd.button() == mouse_button::button1) {
+               if (!update && cmd.button() == mouse_button::button1) {
                        needsUpdate = false;
                        cur.noUpdate();
                }
@@ -1518,7 +1531,16 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
            && oldTopSlice.idx() == cur.idx()
            && !sel // sel is a backup of cur.selection() at the biginning of the function.
            && !cur.selection())
-               cur.noUpdate();
+               // FIXME: it would be better if we could just do this
+               //
+               //if (cur.result().update() != Update::FitCursor)
+               //      cur.noUpdate();
+               // 
+               // But some LFUNs do not set Update::FitCursor when needed, so we
+               // do it for all. This is not very harmfull as FitCursor will provoke
+               // a full redraw only if needed but still, a proper review of all LFUN
+               // should be done and this needsUpdate boolean can then be removed.
+               cur.updateFlags(Update::FitCursor);
        else
                cur.updateFlags(Update::Force | Update::FitCursor);
 }