| commit | 715853139b1cf226b42ea313aa386878af223124 | |
| author | Anders Hartvoll Ruud <andruud@chromium.org> | |
| Thu, 13 Oct 2022 16:03:10 +0000 (13 16:03 +0000) | ||
| committer | moz-wptsync-bot <wptsync@mozilla.com> | |
| Fri, 21 Oct 2022 10:37:11 +0000 (21 10:37 +0000) | ||
| tree | 0a42c6c1ce72e9480b98782bc43995b1010f7db1 | treesnapshot (tar.gz zip) |
| parent | 422a6d201fb94d1ba15f8688e3a8e092830b928b | commitdiff |
Bug 1793484 [wpt PR 36235] - [scroll-animations] Support view-timeline-inset:auto" (reland), a=testonly
Automatic update from web-platform-tests
[scroll-animations] Support view-timeline-inset:auto" (reland)
When the author specifies view-timeline-inset:auto, it is supposed to
use the value from the corresponding scroll-padding property. This CL
implements that behavior.
Since some length utils expect LayoutUnit rather than double, it made
sense to keep values as LayoutUnits where possible to avoid converting
too much back and forth.
Also, it turns out that the default Length() constructor is effectively
"auto", so I had to change the default insets to Length::Fixed(), which
is "0".
Bug: 1317765
Change-Id: I2d6f9fd91fcfac2197bf31ce9c24e9f7f8ba52a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3931554
Auto-Submit: Anders Hartvoll Ruud <andruud@chromium.org>
Reviewed-by: Kevin Ellis <kevers@chromium.org>
Commit-Queue: Kevin Ellis <kevers@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1055948}
--
wpt-commits: b2deaa65f9de3148b8405db172e5ba3c01c482a9
wpt-pr: 36235
Automatic update from web-platform-tests
[scroll-animations] Support view-timeline-inset:auto" (reland)
When the author specifies view-timeline-inset:auto, it is supposed to
use the value from the corresponding scroll-padding property. This CL
implements that behavior.
Since some length utils expect LayoutUnit rather than double, it made
sense to keep values as LayoutUnits where possible to avoid converting
too much back and forth.
Also, it turns out that the default Length() constructor is effectively
"auto", so I had to change the default insets to Length::Fixed(), which
is "0".
Bug: 1317765
Change-Id: I2d6f9fd91fcfac2197bf31ce9c24e9f7f8ba52a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3931554
Auto-Submit: Anders Hartvoll Ruud <andruud@chromium.org>
Reviewed-by: Kevin Ellis <kevers@chromium.org>
Commit-Queue: Kevin Ellis <kevers@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1055948}
--
wpt-commits: b2deaa65f9de3148b8405db172e5ba3c01c482a9
wpt-pr: 36235
| testing/web-platform/tests/scroll-animations/css/view-timeline-inset-animation.html | diffblobblamehistory |