New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prefixed intrinsic size value #16962
Prefixed intrinsic size value #16962
Conversation
Heads up! This PR modifies the following files:
|
All patches have been reviewed by @Manishearth |
@bors-servo: delegate+ |
✌️ @hiikezoe can now approve this pull request |
☔ The latest upstream changes (presumably #16960) made this pull request unmergeable. Please resolve the merge conflicts. |
2fe2916
to
26e19f2
Compare
@bors-servo try |
Prefixed intrinsic size value <!-- Please describe your changes on the following line: --> This is a PR for https://bugzilla.mozilla.org/show_bug.cgi?id=1355402 - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #16788 - [X] These changes do not require tests because it's for stylo <!-- 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/16962) <!-- Reviewable:end -->
… {Min,Max}Length.
So that we can reuse this for non-min-prefixed properties (e.g. width).
auto() and zero() are used in the parser for flex shorthand property.
26e19f2
to
d06af89
Compare
@bors-servo r=Manishearth |
📌 Commit d06af89 has been approved by |
⌛ Testing commit d06af89 with merge f56f4675eedecce37c798f324dad187cd66d70e0... |
💔 Test failed - linux-rel-css |
@bors-servo retry |
…shearth Prefixed intrinsic size value <!-- Please describe your changes on the following line: --> This is a PR for https://bugzilla.mozilla.org/show_bug.cgi?id=1355402 - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #16788 - [X] These changes do not require tests because it's for stylo <!-- 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/16962) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-msvc-dev |
This is a PR for https://bugzilla.mozilla.org/show_bug.cgi?id=1355402
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is