diff options
author | Boris Chiou <boris.chiou@gmail.com> | 2023-06-06 15:26:17 +0200 |
---|---|---|
committer | Oriol Brufau <obrufau@igalia.com> | 2023-06-09 10:22:22 +0200 |
commit | e66bcf2cc589633b212fa70153d823262cf49f36 (patch) | |
tree | 480b2d4ad8376cd41d9d47a63db20896ad8170a6 /components/style/invalidation/stylesheets.rs | |
parent | 84cd22c3e0496e7cf72b474fbee56dd91ddf2246 (diff) | |
download | servo-e66bcf2cc589633b212fa70153d823262cf49f36.tar.gz servo-e66bcf2cc589633b212fa70153d823262cf49f36.zip |
style: Part 8: Hook scroll-timeline rule into Cascade data and use it for CSS animations
We hook the rule into cascade data, and so we can look it up by timeline
name. Now we only use StyleScrollDirection from @scroll-timeline rule.
`source` and `scroll-offsets` are skipped now and use the default values
instead because I'm pretty sure the syntax will be changed in Bug 1733260,
and `scroll-offsets` may be obsolete because the spec proposal intents to
make it be always 0% ~ 100%.
Also, add some reftests for the default `source` and `scroll-offsets`,
and different `orientation`s.
Besides, we disable at-scroll-timeline-start-end.html in Gecko because
we don't support start/end descriptors, and there are too many
intermittents in it.
Differential Revision: https://phabricator.services.mozilla.com/D126452
Diffstat (limited to 'components/style/invalidation/stylesheets.rs')
-rw-r--r-- | components/style/invalidation/stylesheets.rs | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/components/style/invalidation/stylesheets.rs b/components/style/invalidation/stylesheets.rs index fc39b8cca76..1d1111170ed 100644 --- a/components/style/invalidation/stylesheets.rs +++ b/components/style/invalidation/stylesheets.rs @@ -619,11 +619,12 @@ impl StylesheetInvalidationSet { // existing elements. } }, - ScrollTimeline(..) => { - // TODO: Bug 1676784: check if animation-timeline name is referenced. - // Now we do nothing. - }, - CounterStyle(..) | Page(..) | Viewport(..) | FontFeatureValues(..) => { + // TODO: Check if timeline name is referenced, though this might go away in bug 1737918. + ScrollTimeline(..) | + CounterStyle(..) | + Page(..) | + Viewport(..) | + FontFeatureValues(..) => { debug!( " > Found unsupported rule, marking the whole subtree \ invalid." |