Closed Bug 168455 Opened 17 years ago Closed 17 years ago

RTL text with position: relative not reordered on display

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: eyalroz, Assigned: mkaply)

Details

(Keywords: rtl)

Attachments

(2 files)

The attachment file contains two lines of Hebrew text (encoding windows-1255,
e.g. logical, not visual); one of them is rendered properly (i.e. in the
right-aligned line, the first character of the first word appears first on the
right, then the second char of the first word etc). The next line is rendered in
reverse. Apparently when the 'position: relative' is removed the rendering is
proper.
Oh, forgot to mention my build is 2002-09-11-10 (but it was the same for a few
earlier builds I had too).
Confirming on Windows and Linux. 
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All
Summary: Hebrew text rendered end-to-start rather than start-to-end → RTL text with position: relative not reordered on display
Attached patch Patch v.1Splinter Review
Good catch: we weren't including positionedInlineFrame in the list of frames
whose children we need to look at when collecting text from a block element to
pass to the Bidi reordering algorithm.

Adding positionedInlineFrame to the condition solves the problem with this test
case, but maybe there is a simpler and/or more general solution?
Comment on attachment 99147 [details] [diff] [review]
Patch v.1

sr=kin@netscape.com
Attachment #99147 - Flags: superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: zach → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.