diff options
| author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-08-31 11:28:18 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-08-31 11:28:18 -0500 |
| commit | f5a546a1600fe6a6761e3fd6ac1bb6c3f1a503eb (patch) | |
| tree | 451284b4598b47a65c036d479b32b019b56f628e /components/script/layout_wrapper.rs | |
| parent | 59a0be0068cb0e06002d8c4f33491cc67187dbec (diff) | |
| parent | a3af2303d6ca9f40c9c91a11fa7943328350930c (diff) | |
| download | servo-f5a546a1600fe6a6761e3fd6ac1bb6c3f1a503eb.tar.gz servo-f5a546a1600fe6a6761e3fd6ac1bb6c3f1a503eb.zip | |
Auto merge of #12437 - gpoesia:tr_margin_fix, r=pcwalton
Handle row borders in border collapsing logic.
<!-- Please describe your changes on the following line: -->
Handle table row border when collapsing borders for a table row. The row border is combined with the cell's border using the already implemented conflict resolution logic.
This is a screenshot of the following test:
```html
<!doctype html>
<html><body>
<style>
table {
border-collapse: collapse;
}
tr {
border: 1px solid black;
}
</style>
<table>
<tr><td>Lorem</td><td>Ipsum</td><td>Sit</td><td>Dolor</td></tr>
<tr><td>Lorem</td><td>Ipsum</td><td>Sit</td><td>Dolor</td></tr>
<tr><td>Lorem</td><td>Ipsum</td><td>Sit</td><td>Dolor</td></tr>
<tr><td>Lorem</td><td>Ipsum</td><td>Sit</td><td>Dolor</td></tr>
</table>
</body>
</html>
```
<img src="https://dl.dropboxusercontent.com/u/10962672/Screenshots%20Servo/servo_tr_border_collapse.png"/>
The top border is missing, but I think that's a different bug, since it also does not show up when the border is in the cells, and not the rows. Also, when debugging the border collapsing structures, they seem ok (the top border seems to be there). I can look at that bug in a separate issue (or in this one too if you prefer).
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #11527 (github issue number if applicable).
<!-- Either: -->
- [X] These changes do not require tests because I didn't find how to automatically test it (will be happy to provide a test if there's infrastructure for this kind of test already in place).
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
Fixes #11527.
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12437)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/layout_wrapper.rs')
0 files changed, 0 insertions, 0 deletions