299838 - BiDi: Caret disappears outside the end of lines that end with spaces (when such lines are part of an overflowing paragraph)

Status: VERIFIED FIXED

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

Description

[Prognathous]

To reproduce:

1. Open https://bugzilla.mozilla.org/attachment.cgi?id=134102&action=view
2. Put the caret anywhere on the first line (RTL textarea).
2. Press End or press the LeftArrow repeatedly until the caret is at the
end of the line.

Actual Result:
The caret disappears.

Expected result:
The caret should be displayed at the end of the first line. See screenshot of
IE6 handling the same situation:
http://oren.gomen.org/mozilla/textareas6IE6.png

Tested with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2)
Gecko/20050705 Firefox/1.0+

First reported in Bug 207186, comment #16

Prog.

Comment 1

[Uri Bernstein]

Attachment: patch

Handle the RTL case similarly to the way the LTR case is handled. This should
have been included in the fix for bug 19265.

Comment 2

[Prognathous]

*** Bug 89253 has been marked as a duplicate of this bug. ***

Comment 3

[Simon Montagu :smontagu]

This is the same as attachment 41193 [details] [diff] [review] in bug 89253, right? What about the problem
mentioned in bug 89253 comment 2? Has that been fixed by something else in the
meanwhile?

Comment 4

[Uri Bernstein]

(In reply to comment #3)

Yes - it is the same. I didn't realize that patch existed.
I have never seen the problem mentioned in bug 89253 comment 2 - with or without
the patch. So I guess it was fixed a while ago.

Comment 5

[Uri Bernstein]

Simon, Mano - can one of you please check in the patch for me? Thanks.

Comment 6

[Asaf Romano (gone)]

Thanks Uri!

Checking in nsTextFrame.cpp;
/cvsroot/mozilla/layout/generic/nsTextFrame.cpp,v  <--  nsTextFrame.cpp
new revision: 1.513; previous revision: 1.512
done