Skip to content

Commit 2cd4d10

Browse files
committed
[web-animations-1] Tie most in-text issues to GitHub issues
1 parent bdf1ca9 commit 2cd4d10

File tree

1 file changed

+31
-28
lines changed

1 file changed

+31
-28
lines changed

web-animations-1/Overview.bs

Lines changed: 31 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -576,8 +576,8 @@ is calculated as a fixed offset from the |now| timestamp provided each time the
576576
This fixed offset is referred to as the document timeline's <dfn>origin
577577
time</dfn>.
578578

579-
Issue: There must be a better term than &ldquo;origin time&rdquo;&mdash;
580-
it's too similar to &ldquo;time origin&rdquo;.
579+
Issue(2079): There must be a better term than &ldquo;origin time&rdquo;&mdash;
580+
it's too similar to &ldquo;time origin&rdquo;.
581581

582582
Prior to establishing the [=time origin=] for its associated document,
583583
a [=document timeline=] is <a lt="inactive timeline">inactive</a>.
@@ -686,9 +686,10 @@ follows:
686686
<h4 id="responding-to-a-newly-inactive-timeline">Responding to a newly inactive
687687
timeline</h4>
688688

689-
Issue: With the set of timelines defined in this level of this specification,
690-
this situation is not expected to occur. As a result, this section will likely
691-
be moved to a subsequent level of this specification.
689+
Issue(2080): With the set of timelines defined in this level of this
690+
specification, this situation is not expected to occur. As
691+
a result, this section will likely be moved to a subsequent level
692+
of this specification.
692693

693694
When the <a>timeline</a> associated with an <a>animation</a>,
694695
<var>animation</var>, becomes newly <a lt="inactive timeline">inactive</a>,
@@ -1098,11 +1099,11 @@ as CSS Animations [[CSS-ANIMATIONS-1]].
10981099
1. Set the <a>animation start time</a> of <var>animation</var>
10991100
to <var>new start time</var>.
11001101

1101-
Issue(200): If <em>both</em> the <a lt="animation start time">start
1102-
time</a> and <a>hold time</a> are specified, we should
1103-
probably calculate the <a lt="animation start time">start
1104-
time</a> from the <a>hold time</a> instead of using it
1105-
as-is.
1102+
Issue(2073): If <em>both</em> the <a lt="animation start time">start
1103+
time</a> and <a>hold time</a> are specified, we should
1104+
probably calculate the <a lt="animation start time">start
1105+
time</a> from the <a>hold time</a> instead of using it
1106+
as-is.
11061107

11071108
1. <a lt="resolve a Promise">Resolve</a> <var>animation</var>'s <a>current
11081109
ready promise</a> with <var>animation</var>.
@@ -3384,6 +3385,8 @@ Note: this procedure permits overlapping <a>keyframes</a>.
33843385
href='http://lists.w3.org/Archives/Public/public-fx/2013AprJun/0184.html'>section
33853386
4 (Keyframe offsets outside [0, 1]) of minuted discussion from Tokyo
33863387
2013 F2F</a>.
3388+
3389+
<a href="https://github.com/w3c/csswg-drafts/issues/2081">&lt;https://github.com/w3c/csswg-drafts/issues/2081&gt;</a>
33873390
</div>
33883391

33893392

@@ -3927,14 +3930,14 @@ apart from the timing model.
39273930

39283931
</div>
39293932

3930-
Issue: The <code>remove()</code> method can be used to remove an effect from
3931-
either its parent group or animation. Should we keep it in level 1 and define it
3932-
simply as removing the animation from its animation?
3933+
Issue(2082): The <code>remove()</code> method can be used to remove an effect
3934+
from either its parent group or animation. Should we keep it in level 1 and
3935+
define it simply as removing the animation effect from its animation?
39333936

39343937

39353938
<h3 id="the-animationeffecttimingreadonly-interface">The <code>AnimationEffectTimingReadOnly</code> interface</h3>
39363939

3937-
Issue: This interface needs a constructor.
3940+
Issue(2055): This interface needs a constructor.
39383941

39393942
<pre class="idl">
39403943
[Exposed=Window]
@@ -4050,7 +4053,7 @@ The {{AnimationEffectTiming}} interface is a mutable subclass of
40504053
{{AnimationEffectTimingReadOnly}} returned for the <code>timing</code> attribute
40514054
of a mutable <a>animation effect</a> such as {{KeyframeEffect}}.
40524055

4053-
Issue: This interface needs a constructor.
4056+
Issue(2055): This interface needs a constructor.
40544057

40554058
<pre class='idl'>
40564059
[Exposed=Window]
@@ -4373,8 +4376,8 @@ interface KeyframeEffect : KeyframeEffectReadOnly {
43734376
Attributes must be assigned in the order in which they appear in the
43744377
{{AnimationEffectTimingReadOnly}} interface.
43754378

4376-
Issue: Make a constructor for {{AnimationEffectTimingReadOnly}} and call
4377-
that here.
4379+
Issue(2055): Make a constructor for {{AnimationEffectTimingReadOnly}}
4380+
and call that here.
43784381

43794382
1. If <var>options</var> is a {{KeyframeEffectOptions}} object,
43804383
assign the {{KeyframeEffectReadOnly/iterationComposite}}, and
@@ -5562,16 +5565,16 @@ partial interface Document {
55625565
changes to animation such as changes to animation-related style properties
55635566
that have yet to be processed.
55645567
5565-
Issue: Both this method and {{Animatable/getAnimations()}} on the
5566-
{{Animatable}} interface require retaining forwards-filling
5567-
<a>animation effects</a> and their <a>animations</a>
5568-
such that a document that
5569-
repeatedly produces forwards-filling animations will consume memory
5570-
in an unbounded fashion.
5571-
We may need to revise this definition (previously these methods
5572-
only returned animations whose <a>target effect</a> was
5573-
<a>current</a>) or provide a loophole for implementations to discard
5574-
old animations in such conditions.
5568+
Issue(2054): Both this method and {{Animatable/getAnimations()}} on the
5569+
{{Animatable}} interface require retaining forwards-filling
5570+
<a>animation effects</a> and their <a>animations</a>
5571+
such that a document that
5572+
repeatedly produces forwards-filling animations will consume
5573+
memory in an unbounded fashion.
5574+
We may need to revise this definition (previously these methods
5575+
only returned animations whose <a>target effect</a> was
5576+
<a>current</a>) or provide a loophole for implementations to
5577+
discard old animations in such conditions.
55755578
55765579
</div>
55775580
@@ -5792,7 +5795,7 @@ Animations content.
57925795
If provided, this behavior SHOULD be achieved by adjusting the <a>time
57935796
values</a> of any <a>timelines</a> that track wallclock time.
57945797
5795-
Issue: Is this at odds with those <a>time values</a> being relative to
5798+
Issue(2083): Is this at odds with those <a>time values</a> being relative to
57965799
<code>navigationStart</code> and with <code>requestAnimationFrame</code>
57975800
using the same time as <code>document.timeline.currentTime</code>?
57985801

0 commit comments

Comments
 (0)