Skip to content

Commit 77d0801

Browse files
committed
Simplify the 'Extensions to the DocumentTimeline interface' section
1 parent 9a8316d commit 77d0801

1 file changed

Lines changed: 14 additions & 27 deletions

File tree

Overview.bs

Lines changed: 14 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,20 @@ active <strong>and</strong> all the other criteria for the timeline becoming
164164
active are met. When the trigger becomes inactive, the timeline becomes inactive
165165
as well.
166166

167+
## Extensions to the {{DocumentTimeline}} interface ## {#document-timeline-interface}
168+
169+
<pre class="idl">
170+
partial dictionary DocumentTimelineOptions {
171+
AnimationTimelineTrigger trigger;
172+
};
173+
[Constructor(DocumentTimelineOptions options)]
174+
partial interface DocumentTimeline {
175+
// trigger attribute inherited from AnimationTimeline
176+
};
177+
</pre>
178+
179+
Issue: Need to add {{DocumentTimelineOptions}} to [[WEB-ANIMATIONS-1]]
180+
167181
## Scroll Triggers ## {#scroll-triggers}
168182

169183
### The {{ScrollDirection}} enumeration ### {#scrolldirection-enumeration}
@@ -308,33 +322,6 @@ element.
308322
</div>
309323
</div>
310324

311-
## Extensions to the {{DocumentTimeline}} interface ## {#document-timeline-interface}
312-
313-
<pre class="idl">
314-
partial dictionary DocumentTimelineOptions {
315-
ScrollTrigger trigger;
316-
};
317-
[Constructor(DocumentTimelineOptions options)]
318-
partial interface DocumentTimeline {
319-
readonly attribute ScrollTrigger trigger;
320-
};
321-
</pre>
322-
323-
Issue: We should add the DocumentTimelineOptions to the [[WEB-ANIMATIONS-1]].
324-
325-
Issue: Do we need to repeat the attribute here? It is already listed in [[#extensions-to-animationtimeline]].
326-
327-
The {{DocumentTimeline}} is defined in [[WEB-ANIMATIONS-1]].
328-
329-
If we specified the trigger, this DocumentTimeline will perform following behaviors.
330-
1. If the ScrollTriggerKind of specified options is "offset":
331-
1. DocumentTimeline will return current document timeline time after exceed specified offset.
332-
1. If the ScrollTriggerKind of specified options is "range":
333-
1. DocumentTimeline wil lreturn current document timeline time when scroll offset is within specified range.
334-
335-
ISSUE: Is it necessary to specify the above, or does it fall out naturally from the definitions of
336-
{{ScrollTrigger}}, {{AnimationTimeline/trigger}}, and [[web-animations#inactive-timeline]]?
337-
338325
## The 'scroll-container-name' property ## {#scroll-container-name}
339326

340327
<pre class='propdef'>

0 commit comments

Comments
 (0)