Skip to content

Commit 1bd64bc

Browse files
committed
[scroll-animations-1][editorial] Minor wording tweaks
1 parent 9fd9016 commit 1bd64bc

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

scroll-animations-1/rewrite.bs

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ spec: cssom-view-1; type: dfn;
202202
</pre>
203203

204204
A {{ScrollTimeline}} is an {{AnimationTimeline}}
205-
that specifies a [=scroll progress timeline=].
205+
that represents a [=scroll progress timeline=].
206206
It can be passed to
207207
the {{Animation}} constructor or the {{Animatable/animate()}} method
208208
to link the animation to a [=scroll progress timeline=].
@@ -213,9 +213,9 @@ spec: cssom-view-1; type: dfn;
213213
whose scroll position drives the progress of the timeline.
214214

215215
: <dfn>axis</dfn>
216-
:: Specifies the axis of scrolling
216+
:: The axis of scrolling
217217
that drives the progress of the timeline.
218-
See <<axis>>, above.
218+
See value definitions for <<axis>>, above.
219219
</dl>
220220

221221
<dl class="constructors">
@@ -242,11 +242,11 @@ spec: cssom-view-1; type: dfn;
242242

243243
Inherited attributes:
244244
<dl>
245-
: {{AnimationTimeline/currentTime}} (inherited from {{AnimationTimeline}}
245+
: {{AnimationTimeline/currentTime}} (inherited from {{AnimationTimeline}})
246246
:: Represents the scroll progress of the [=scroll container=]
247247
as a percentage CSSUnitValue,
248248
with 0% representing its startmost scroll position
249-
(using the [=writing mode=] of the [=scroll container=].
249+
(in the [=writing mode=] of the [=scroll container=]).
250250
Null when the timeline is [=timeline inactive phase|inactive=].
251251
</dl>
252252

@@ -256,10 +256,13 @@ spec: cssom-view-1; type: dfn;
256256
Is this what we want?
257257
<!-- turn this into a note instead of a question once resolved -->
258258

259+
ISSUE: Add a note about whether {{AnimationTimeline/currentTime}}
260+
can be negative or > 100%.
261+
259262
If the {{ScrollTimeline/source}} of a {{ScrollTimeline}}
260263
is an element whose [=principal box=] does not exist
261264
or is not a [=scroll container=],
262-
then its {{AnimationTimeline/phase}} is the [=timeline inactive phase=].
265+
then the {{ScrollTimeline}} is in the [=timeline inactive phase=].
263266
It is otherwise in the [=timeline active phase|active=] phase.
264267

265268
A {{ScrollTimeline}}’s {{EffectTiming/duration}} is 100%.

0 commit comments

Comments
 (0)