Closed
Bug 168455
Opened 22 years ago
Closed 22 years ago
RTL text with position: relative not reordered on display
Categories
(Core :: Layout: Text and Fonts, defect)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
People
(Reporter: eyalroz1, Assigned: mkaply)
Details
(Keywords: rtl)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
dbaron
:
review+
kinmoz
:
superreview+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•22 years ago
|
||
Oh, forgot to mention my build is 2002-09-11-10 (but it was the same for a few earlier builds I had too).
Comment 2•22 years ago
|
||
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
Comment 3•22 years ago
|
||
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 4•22 years ago
|
||
Comment on attachment 99147 [details] [diff] [review] Patch v.1 r=dbaron
Attachment #99147 -
Flags: review+
Comment on attachment 99147 [details] [diff] [review] Patch v.1 sr=kin@netscape.com
Attachment #99147 -
Flags: superreview+
Comment 6•22 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 7•17 years ago
|
||
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.
Description
•