Summary: | Collapsed borders appear on the wrong side or on the wrong cell in RTL tables | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||||||
Component: | Tables | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | VERIFIED FIXED | ||||||||||
Severity: | Normal | Keywords: | HasReduction | ||||||||
Priority: | P2 | ||||||||||
Version: | 420+ | ||||||||||
Hardware: | Mac | ||||||||||
OS: | OS X 10.4 | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 6838 | ||||||||||
Attachments: |
|
Description
mitz
2006-01-28 05:11:11 PST
Created attachment 6045 [details]
Testcase
Firefox renders this correctly.
Created attachment 6050 [details]
Proposed patch
Comment on attachment 6050 [details]
Proposed patch
I think it's a little inelegant to have that inLastColumn boolean and those "rtl ? !inLastColumn : (col() > 0)" clauses. Instead, could we just compute an appropriate boolean up front? Something like "righmostColumn" or "leftmostColumn"?
Looks great to me. r=me
Created attachment 6072 [details]
Updated patch
Replaced the inLastColumn booleans with leftmostColumn/rightmostColumn per Darin's suggestion.
Comment on attachment 6072 [details]
Updated patch
Looks great. I think it's ready to land, but I do have 2 more ideas to improve it. I'll probably just do these if I land it:
1) cellPrevious and cellNext should be named cellBefore and cellAfter to match the names cellAbove and cellBelow
2) code to compute leftmostColumn and rightmostColumn should have more closely-matching sides of the if, no need to use an if statement in the RTL case
|