diff options
author | bors-servo <infra@servo.org> | 2023-06-29 14:14:55 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-29 14:14:55 +0200 |
commit | a07ff91cfbeccf77b5744747107b0729535b8d69 (patch) | |
tree | 619ad528587e55ab1590f7afd9d129a00056bf70 /components/script/dom/bindings/codegen/parser/tests | |
parent | a725380db0b9fba31993409f7d0f4b2d11ca8f7d (diff) | |
parent | 6b2bbdd02d88cc45d54e331537c10942b6cde990 (diff) | |
download | servo-a07ff91cfbeccf77b5744747107b0729535b8d69.tar.gz servo-a07ff91cfbeccf77b5744747107b0729535b8d69.zip |
Auto merge of #29946 - Loirooriol:clearance-option, r=mrobinson
Layout 2020: implement clearance as Option<Length>
Clearance was implemented as a Length, where zero meant no clearance. However, having a clearance of 0px should be different than having no clearance, since the former can still prevent margin collapse.
This patch keeps the existing behavior, so it won't be possible to get a clearance of Some(Length::zero()), but it prepares the terrain for a follow-up to fix calculate_clearance to return the proper thing.
<!-- Please describe your changes on the following line: -->
---
<!-- 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 do not require tests because there should be no change in behavior
<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
Diffstat (limited to 'components/script/dom/bindings/codegen/parser/tests')
0 files changed, 0 insertions, 0 deletions