Dov Feldstern wrote:
Abdelrazak Younes wrote:
Stefan Schimanski wrote:
I have fixed the logic now in my patch to equal that of LyX 1.4
exactly (see attached).
I've tested this last patch and all seems fine to me WRT cursor
movement. The code cleanup is good too.
Abdel.
Do you not get a crash with this version (the same RTL crash you
reported a couple of weeks ago)?
I just tested again. With the latest version of the patch
(http://permalink.gmane.org/gmane.editors.lyx.devel/85600) I'm getting a
crash. To reproduce: (1) make sure RTL option is turned on; (2) start
typing (with spaces, just normal typing) until the paragraph breaks to a
new line. Start moving backwards (LEFT, assuming LTR). When you hit the
beginning of the line and try moving back again, you get a crash. This
is the same issue reported by Abdel two weeks ago. Copying the
conditions from 1.4 just undoes my fix for that issue. Therefore, I
propose to stick with the previous version
(http://permalink.gmane.org/gmane.editors.lyx.devel/85585). I reported
this yesterday
(http://permalink.gmane.org/gmane.editors.lyx.devel/85611), but didn't
get any reply...
Dov
- Re: Text::cursor* clean up, target_x, right display m... Dov Feldstern
-
