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)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
People
(Reporter: uriber, Assigned: masayuki)
References
Details
(Keywords: regression, rtl)
Attachments
(2 files)
147 bytes,
text/html
|
Details | |
1.09 KB,
patch
|
rbs
:
review+
rbs
:
superreview+
rbs
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•19 years ago
|
||
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.
Assignee | ||
Comment 2•19 years ago
|
||
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)
Assignee | ||
Updated•19 years ago
|
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+
Assignee | ||
Comment 4•19 years ago
|
||
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.
Comment 6•17 years ago
|
||
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.
Description
•