328656 - In RTL textarea, using left arrow to move the caret from a blank line into a line starting with LTR text moves the caret to the wrong position

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Uri Bernstein]

In an RTL textarea, if the caret is on a blank line, and the next line starts with LTR text, pressing left-arrow moves the caret to a position after the LTR text on the next line, instead of to the beginning of that line.
Similarly, if the caret is on a blank line, and the previous line ends with LTR text, pressing right-arrow moves the caret to a position before the LTR text on the previous line, instead of to the beginning of that line.

This does not affect LTR textareas.

This is (somewhat surprisingly) not a regression, at least not since Fx 1.0.

Testcase coming up.

Comment 1

[Uri Bernstein]

Attachment: testcase

(instructions included within).

Comment 2

[Simon Montagu :smontagu]

This may be a dupe; at least I remember trying to find a fix for it in the past :)

Comment 3

[Simon Montagu :smontagu]

Note also that if you place the caret on line 4 of the test case and press End and then right arrow, the caret skips completely over the following blank line.

Comment 4

[Simon Montagu :smontagu]

s/right/left/ in the previous comment

Comment 5

[Uri Bernstein]

(In reply to comment #3)
> Note also that if you place the caret on line 4 of the test case and press End
> and then right arrow, the caret skips completely over the following blank line.

This issue is actually related to bug 303884 (it's fixed by the first patch on that bug). I hope to get around to writing a decent patch for that bug, once the dust on bug 299065 settles a bit.

Comment 6

[Uri Bernstein]

Fixing bug 303884 fixed the "right arrow to get to previous line case", and also the issue mentioned in comment #3. However the issue with using left-arrow to move from an empty line to a line starting with LTR text remains.

Comment 7

[Uri Bernstein]

Attachment: testcase

The fix for bug 330815 fixed the problem for the original testcase.
However, the same problem now exists in the (less common) case where the line on the "other" side of the blank line also ends/starts with reverse-direction text.

This problem is symmetrical, i.e., it affects LTR textareas as well.

This is now similar to bug 331444, and will hopefully be fixed by fixing that bug.

Comment 8

[Uri Bernstein]

This (both testcases) should be fixed by the checkin for bug 331444.

Comment 9

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).