diff options
| author | Matt Brubeck <mbrubeck@limpet.net> | 2015-05-08 14:24:34 -0700 |
|---|---|---|
| committer | Matt Brubeck <mbrubeck@limpet.net> | 2015-05-08 21:29:02 -0700 |
| commit | e36a498cfbafe61a3a0b02541e2f49674f88f757 (patch) | |
| tree | 33ef22eb06a6fc31f50d7d1cd51e17d8471d8cd2 /components/layout/table.rs | |
| parent | a052c53d2e79a0024b62087815eb5d7e4e56d5c9 (diff) | |
| download | servo-e36a498cfbafe61a3a0b02541e2f49674f88f757.tar.gz servo-e36a498cfbafe61a3a0b02541e2f49674f88f757.zip | |
Fix direction of columns in mixed LTR/RTL tables.
Table columns should be layed out according to the 'direction' property of the
table flow, regardless of the 'direction' property of any table-row,
table-rowgroup, etc. flows.
This fixes a number of the `direction-applies-to-*` tests in the CSS2.1 test
suite.
This also simplifies `propagate_column_inline_sizes_to_child` by separating
the code used for table cells from the code for non-cell flows.
r? @pcwalton
Diffstat (limited to 'components/layout/table.rs')
| -rw-r--r-- | components/layout/table.rs | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/components/layout/table.rs b/components/layout/table.rs index 72af04817f2..3e73bc8329a 100644 --- a/components/layout/table.rs +++ b/components/layout/table.rs @@ -473,19 +473,16 @@ impl Flow for TableFlow { inline_end_content_edge, content_inline_size, |child_flow, - child_index, - content_inline_size, + _child_index, + _content_inline_size, writing_mode, - inline_start_margin_edge| { + _inline_start_margin_edge, + _inline_end_margin_edge| { table_row::propagate_column_inline_sizes_to_child( child_flow, - child_index, - content_inline_size, writing_mode, column_computed_inline_sizes, - &spacing_per_cell, - &None, - inline_start_margin_edge); + &spacing_per_cell); if child_flow.is_table_row() { let child_table_row = child_flow.as_table_row(); child_table_row.populate_collapsed_border_spacing( |