Andre Poenitz wrote:
On Mon, May 14, 2007 at 08:03:41PM -0000, [EMAIL PROTECTED] wrote:
Author: broider
Date: Mon May 14 22:03:40 2007
New Revision: 18323
URL: http://www.lyx.org/trac/changeset/18323
Log:
add comments and rtl support for cursor positioning problem
Modified:
lyx-devel/trunk/src/Text2.cpp
Modified: lyx-devel/trunk/src/Text2.cpp
URL: http://www.lyx.org/trac/file/lyx-devel/trunk/src/Text2.cpp?rev=18323
==============================================================================
--- lyx-devel/trunk/src/Text2.cpp (original)
+++ lyx-devel/trunk/src/Text2.cpp Mon May 14 22:03:40 2007
@@ -993,8 +993,16 @@
int x = cur.targetX();
cur.setTargetX();
- if (cur.pos() != pm.rows()[row].endpos() || x < cur.targetX())
+ // We want to keep the x-target on subsequent up movements
+ // that cross beyond the end of short lines. Thus a special
+ // handling when the cursor is at the end of line: Use the new
+ // x-target only if the old one was before the end of line.
+ if (cur.pos() != pm.rows()[row].endpos()
+ || (!cur.isRTL() && x < cur.targetX())
+ || (cur.isRTL() && x > cur.targetX())) {
+
x = cur.targetX();
+ }
Can anybody remind me why we change targetX for cursor up/down at all?
to memorize cursor position?
Abdel.
