Bug 88110 - Prepare table collapsed border computation to support mixed directionality on row group
Summary: Prepare table collapsed border computation to support mixed directionality on...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tables (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Julien Chaffraix
URL:
Keywords:
Depends on:
Blocks: 87900
  Show dependency treegraph
 
Reported: 2012-06-01 11:00 PDT by Julien Chaffraix
Modified: 2012-06-01 13:15 PDT (History)
7 users (show)

See Also:


Attachments
Proposed preparation. Comments on naming more than welcome. (15.04 KB, patch)
2012-06-01 11:19 PDT, Julien Chaffraix
no flags Details | Formatted Diff | Diff
Patch for landing (15.00 KB, patch)
2012-06-01 12:30 PDT, Julien Chaffraix
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Chaffraix 2012-06-01 11:00:57 PDT
The code doesn't handle mixed directionality inside a table and will fail pretty badly if it is involved. CSS 2.1 and CSS 3 writing-modes however allow the use of mixed directionality.

The difficulty with mixed directionality is that the start / end borders don't map strictly anymore: the table start border will be the section's end border if the table and section have mixed directionality.

To account for that, we need new methods to properly get the adjoining borders regardless of the directions.
Comment 1 Julien Chaffraix 2012-06-01 11:19:37 PDT
Created attachment 145340 [details]
Proposed preparation. Comments on naming more than welcome.
Comment 2 Julien Chaffraix 2012-06-01 11:28:17 PDT
CC'ing some people who could comment on the naming used.
Comment 3 Ojan Vafai 2012-06-01 12:06:51 PDT
Comment on attachment 145340 [details]
Proposed preparation. Comments on naming more than welcome.

View in context: https://bugs.webkit.org/attachment.cgi?id=145340&action=review

> Source/WebCore/ChangeLog:14
> +        This change add the concept of adjoining borders in the table direction so that we can safely

s/add/adds

> Source/WebCore/rendering/RenderTableSection.h:127
> +    // These methods handle mixed directions between table and sections.

Meh. This is a what comment. That will be clear once you do your followup patch. No need to add a comment saying it.
Comment 4 Julien Chaffraix 2012-06-01 12:30:13 PDT
Created attachment 145356 [details]
Patch for landing
Comment 5 WebKit Review Bot 2012-06-01 13:15:19 PDT
Comment on attachment 145356 [details]
Patch for landing

Clearing flags on attachment: 145356

Committed r119272: <http://trac.webkit.org/changeset/119272>
Comment 6 WebKit Review Bot 2012-06-01 13:15:24 PDT
All reviewed patches have been landed.  Closing bug.