Fix for caret not moving

jEdit is a programmer's text editor written in Java.

Brought to you by: daleanson, ezust, grepppo, k_satoda, and 5 others

#86 Fix for caret not moving

Milestone: for 4.4.x

Status: closed-accepted

Owner: Alan Ezust

Labels: None

Priority: 5

Updated: 2011-10-05

Created: 2011-08-28

Creator: Matthieu Casanova

Private: No

please merge rev 19847

Fixed a corner case when trying to move caret during a buffer operation (loading, transaction, undo) prevent to move the caret forever.

The easiest way is of course to reproduce it while loading buffer : press ctrl+End while the buffer is loading, and it cannot move anymore.

This can also be merged in 4.3

Discussion

Alan Ezust - 2011-08-30

milestone: 1360914 --> 1012693

status: open --> open-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Alan Ezust - 2011-08-30

Ignoring the changes to the .iml files in that revision, I'm assuming those were a mistake.

Committed r19874 to 4.4.x branch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Alan Ezust - 2011-09-06

un-merged rev# 19935

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Alan Ezust - 2011-09-06

milestone: 1012693 -->

status: open-accepted --> closed-rejected
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Matthieu Casanova - 2011-09-09

status: closed-rejected --> open
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Matthieu Casanova - 2011-09-09

Same problem but different fix, revision 19947

This time it is possible to type out of the screen and the caret will be made visible as usual.
I hope there is no other side effect

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Björn Kautler - 2011-09-24

Matthieu, does this mean r19947 should be merged instead of r19847?
If yes, then please create a separate merge request for 4.3.x.
If not and it meant that this merge request should be void, then please set it to "Rejected".

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Björn Kautler - 2011-09-24

milestone: --> for 4.4.x
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Matthieu Casanova - 2011-09-30

Yes I think this time the fix in rev 19947 is the good one and should be merged

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Matthieu Casanova - 2011-09-30

Finally do NOT merge in 4.3.x

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Alan Ezust - 2011-10-05

assigned_to: nobody --> ezust

status: open --> closed-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Alan Ezust - 2011-10-05

wow, this is just a 1-liner compared to the one before which was quite big and in a different file!

Merged rev#19947 into 4.4.x branch, as rev# 20052.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.