Closed Bug 330268 Opened 19 years ago Closed 19 years ago

RTL characters typed into a textarea after a long unbreakable string don't appear on screen until scrolled

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: uriber, Assigned: masayuki)

References

Details

(Keywords: regression, rtl)

Attachments

(2 files)

If a textarea contains a long unbreakable string, causing a horizontal scrollbar to appear, then typing RTL characters after that string results in only the first RTL character appearing. scrolling the textarea to the right reveals the other characters.

This appeared on the trunk on 2006-03-08, so I naturally suspected bug 319930. However, backing out the patch for that bug didn't fix it, whereas backing out the patch for bug 164700 did fix it.

I'll attach a testcase soon.
Attached file testcase (deleted) —
You'll need an RTL (Hebrew or Arabic) keyboard layout for this.

1. Move the caret to the end of the text (after the "z").
2. Type a space (this is important!)
3. Type some RTL (Hebrew or Arabic) characters. Notice that the first character you type appears on the second line. However, subsequent characters don't appear anywhere.
4. Scroll to the right to reveal the missing characters.
Attached patch Patch rv1.0 (deleted) — Splinter Review
This is not a regression, because current code doesn't support INLINE_BREAK_AFTER.
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Attachment #214866 - Flags: superreview?(rbs)
Attachment #214866 - Flags: review?(rbs)
Attachment #214866 - Flags: approval-branch-1.8.1?(rbs)
OS: MacOS X → All
Hardware: Macintosh → All
Comment on attachment 214866 [details] [diff] [review]
Patch rv1.0

Let's bake it on the trunk a few days before landing on the branch.
Attachment #214866 - Flags: superreview?(rbs)
Attachment #214866 - Flags: superreview+
Attachment #214866 - Flags: review?(rbs)
Attachment #214866 - Flags: review+
Attachment #214866 - Flags: approval-branch-1.8.1?(rbs)
Attachment #214866 - Flags: approval-branch-1.8.1+
checked-in to trunk.

rbs:
> before landing on the branch.

We need to fix bug 329987 too before landing on the branch.
Please review it.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
I am aware of that. I want to experiment that bug/patch myself. But I am still blocked by my broken tree from bug 329209.
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: