>>>>> "Martin" == Martin Vermeer <[EMAIL PROTECTED]> writes:

Martin> I remember seeing this too. Attached a patch for two places
Martin> where full screen updates would be reasonable but are not
Martin> being done. The text3 one probably addresses Andre's problem.

Could you elaborate on why these two chunks make sense?

@@ -1051,9 +1054,9 @@ void LyXText::dispatch(LCursor & cur, Fu
                        CursorSlice old = bvcur.top();
 
                        int const wh = bv->workHeight();
-                       int const y = std::max(0, std::min(wh - 1, cmd.y));
+                       //int const y = std::max(0, std::min(wh - 1, cmd.y));
 
-                       setCursorFromCoordinates(cur, cmd.x, y);
+                       //setCursorFromCoordinates(cur, cmd.x, y);
                        cur.x_target() = cmd.x;
                        if (cmd.y >= wh)
                                cursorDown(cur);
@@ -1128,7 +1131,6 @@ void LyXText::dispatch(LCursor & cur, Fu
 
                cur.resetAnchor();
                moveCursor(cur, false);
-               bv->updateScrollbar();
                break;
        }
 
Concerning the first one, if you know what you are doing, you should
just delete the lines.

Martin> The insetcollapsable one is about the "jumping" of the inset
Martin> label from inlined to above, causing "turds" lying around.

This one looks good, except that I am not sure coordcache.h is needed.

Martin> (And no, a similar patch for Delete is not needed. Don't ask.)

OK, I won't ask :)

JMarc

Reply via email to