Skip to content

Commit

Permalink
layout: <th> should have text-align: center when the child of an …
Browse files Browse the repository at this point in the history
…element with `text-align: initial` (servo#33427)

This behavir is enabled via a special CSS value that we can share with
Gecko.

Signed-off-by: Martin Robinson <[email protected]>
Co-authored-by: Oriol Brufau <[email protected]>
  • Loading branch information
mrobinson and Loirooriol authored Sep 12, 2024
1 parent 219a2f2 commit db0aee6
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 26 deletions.
28 changes: 14 additions & 14 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion resources/user-agent.css
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,10 @@ table {
text-indent: initial;
}
td, th { padding: 1px; }
th { font-weight: bold; }
th {
font-weight: bold;
text-align: -moz-center-or-inherit;
}

thead, tbody, tfoot, table > tr { vertical-align: middle; }
tr, td, th { vertical-align: inherit; }
Expand Down
2 changes: 0 additions & 2 deletions tests/wpt/meta/css/css-tables/th-text-align.html.ini

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -113,12 +113,6 @@
[table td align attribute justify is correct]
expected: FAIL

[table th align attribute center is correct]
expected: FAIL

[table th align attribute middle is correct]
expected: FAIL

[table th align attribute left is correct]
expected: FAIL

Expand All @@ -145,6 +139,3 @@

[table cellpadding attribute is correct]
expected: FAIL

[th default align attribute is center]
expected: FAIL

0 comments on commit db0aee6

Please sign in to comment.