Bug 5980

Summary: Bidi properties of an inline container whose first child is an inline container are ignored
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Dave Hyatt <hyatt>
Status: VERIFIED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
testcase
none
Adjust embedding at inline containers' boundaries instead of inside hyatt: review+

mitz
Reported 2005-12-06 11:42:23 PST
Summary: Embedding levels aren't adjusted according to the direction and unicode-bidi properties of an inline container if its first child is an inline container. Similarly, embedding levels aren't reset after an inline container whose last child is an inline container -- in both cases, bidiNext() doesn't call embed(). To reproduce: open the attached testcase in Safari. Expected: all lines to read "ABCDEFGHI" (some letters may appear in italics. Firefox renders as expected. Actual: the letters are in the wrong order on the 2nd, 3rd and 5th lines.
Attachments
testcase (592 bytes, text/html)
2005-12-06 11:43 PST, mitz 		no flags
Details
Adjust embedding at inline containers' boundaries instead of inside (13.72 KB, patch)
2005-12-07 07:38 PST, mitz 		hyatt: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2005-12-06 11:43:42 PST
Created attachment 4983 [details] testcase
mitz
Comment 2 2005-12-07 07:38:21 PST
Created attachment 4989 [details] Adjust embedding at inline containers' boundaries instead of inside
Dave Hyatt
Comment 3 2005-12-07 14:52:17 PST
Comment on attachment 4989 [details] Adjust embedding at inline containers' boundaries instead of inside r=me. I even understood it!
Note You need to log in before you can comment on or make changes to this bug.