RESOLVED FIXED 8387
border-left/padding-left do not show up on multiline rtl inline boxes 
https://bugs.webkit.org/show_bug.cgi?id=8387
Summary border-left/padding-left do not show up on multiline rtl inline boxes
Sam Weinig
Reported 2006-04-14 12:38:54 PDT
No border or padding is applied for the left on a multi-line rtl inline box flow. I am not shore what the correct behavior is supposed to be, but the right side is painted at the begining of the flow (in the ltr context the right side is at the end) so I assume the left should be at the end.
Attachments
test case (2.61 KB, text/html)
2006-04-14 12:41 PDT, Sam Weinig 		no flags
Details
patch (5.68 KB, patch)
2006-04-15 13:23 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
test case without bidi overide (2.70 KB, text/html)
2006-04-17 14:50 PDT, Sam Weinig 		no flags
Details
test case with inner spans going in opposite directions of there parent (2.93 KB, text/html)
2006-04-17 14:51 PDT, Sam Weinig 		no flags
Details
test case with opposite direction inner spans and no bidi overide (2.80 KB, text/html)
2006-04-17 14:52 PDT, Sam Weinig 		no flags
Details
test case using hebrew text (1.91 KB, text/html)
2006-04-17 14:53 PDT, Sam Weinig 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2006-04-14 12:41:16 PDT
Created attachment 7708 [details] test case This test case shows both ltr and rtl of the same text documenting both single and multiline inline boxes.
Joost de Valk (AlthA)
Comment 2 2006-04-14 12:49:26 PDT
I confirm the behavior is wrong, this sounds like one mitz find interesting.
Sam Weinig
Comment 3 2006-04-15 13:23:58 PDT
Created attachment 7733 [details] patch Simple 1 line fix with new layout test.
Sam Weinig
Comment 4 2006-04-17 14:48:54 PDT
While the patch does fix the functionality shown in the test case and cause no regressions that I could find, in my testing I found that the issue persists in a lot of other corner cases. I am going to attach a couple more test cases that demonstrate.
Sam Weinig
Comment 5 2006-04-17 14:50:07 PDT
Created attachment 7779 [details] test case without bidi overide
Sam Weinig
Comment 6 2006-04-17 14:51:10 PDT
Created attachment 7780 [details] test case with inner spans going in opposite directions of there parent
Sam Weinig
Comment 7 2006-04-17 14:52:05 PDT
Created attachment 7781 [details] test case with opposite direction inner spans and no bidi overide
Sam Weinig
Comment 8 2006-04-17 14:53:30 PDT
Created attachment 7782 [details] test case using hebrew text
Dave Hyatt
Comment 9 2006-05-07 15:20:32 PDT
Comment on attachment 7733 [details] patch r=me
Alexey Proskuryakov
Comment 10 2006-05-13 10:14:47 PDT
After applying the change to rendering/InlineFlowBox.cpp, the results don't really look like they are supposed to be (multiline boxes only get one colored border).
Alexey Proskuryakov
Comment 11 2006-05-13 10:22:29 PDT
Comment on attachment 7733 [details] patch Clearing the review flag for now.
Alexey Proskuryakov
Comment 12 2006-05-14 09:35:22 PDT
The test case was affected by bug 8392, so I have slightly modified it before landing. It should work OK in run-webkit-tests now, but it still depends on the window width, so it may fail when tried manually.
Note You need to log in before you can comment on or make changes to this bug.