-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
CSS Grid: percentage sizing fixes #34948
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
Conversation
aab4a20 to
7378d94
Compare
5666109 to
d408f28
Compare
|
🔨 Triggering try run (#14186207551) for Linux (WPT) |
|
Test results for linux-wpt from try job (#14186207551): Flaky unexpected result (28)
Stable unexpected results that are known to be intermittent (12)
Stable unexpected results (1)
|
|
|
d408f28 to
fb6ff2a
Compare
|
🔨 Triggering try run (#14187080489) for Linux (WPT) |
40da553 to
3a0695e
Compare
|
🔨 Triggering try run (#14187169647) for Linux (WPT) |
f1a3d0b to
818c055
Compare
|
🔨 Triggering try run (#14187345951) for Linux (WPT) |
|
Test results for linux-wpt from try job (#14187169647): Flaky unexpected result (19)
Stable unexpected results that are known to be intermittent (19)
|
|
✨ Try run (#14187169647) succeeded. |
|
🔨 Triggering try run (#15353106051) for Linux (WPT) |
|
Test results for linux-wpt from try job (#15353106051): Flaky unexpected result (18)
Stable unexpected results that are known to be intermittent (19)
|
|
✨ Try run (#15353106051) succeeded. |
286f4a6 to
59f9e2a
Compare
|
🔨 Triggering try run (#15518627535) for Linux (WPT) |
|
Test results for linux-wpt from try job (#15518627535): Flaky unexpected result (27)
Stable unexpected results that are known to be intermittent (15)
|
|
✨ Try run (#15518627535) succeeded. |
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
Signed-off-by: Nico Burns <nico@nicoburns.com>
59f9e2a to
cc3a464
Compare
Depends on: - servo/stylo#104 - #34926 - #34927 - #34948 In addition to that a `resolve_calc_value` function has been added which resolves calc values during the layout process once a percentage resolution basis is available. ~~There is 1 newly failing test and 1 newly failing subtest here.~~ These issues have now been fixed. There are 8 new subtest failures in `css/css-grid/grid-definition/grid-minimum-contribution-with-percentages.html`. These are genuine failures, but are unrelated to the calc implementation. The calc implementation is just exposing a pre-existing bug around percentage resolution that also now correctly also applies to `calc()` values containing percentages. The fix for would best be done in a followup as it requires teaching Taffy about "compressible replaced elements". (update: I have this [implemented](#34948) but in the interest of keeping it to one feature per PR I have not included it here) --- <!-- 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. It doesn't like the git dependency on Taffy. - [x] There are tests for these changes OR --------- Signed-off-by: Nico Burns <nico@nicoburns.com>
This applies some fixes for CSS Grid percentage sizing. These fixes are mostly within Taffy, but there are some changes in Servo to allow it to communicate whether an item is replaced to Taffy.
It also updates Taffy to v0.8.0. Taffy has switched to a tagged pointer representation of length/size styles. Much of the diff here is updating Servo's type conversion code to use the new representation.
./mach build -ddoes not report any errors./mach test-tidydoes not report any errors