Editorial: fixed some bikeshed errors and warnings

Author: ydaniv

web-animations-2/Overview.bs

@@ -114,6 +114,7 @@ urlPrefix: https://drafts.csswg.org/web-animations-1/; type: dfn; spec: web-anim
     text: iteration progress
     text: iteration start
     text: keyframe effect
+    text: monotonically increasing
     text: not animatable
     text: overall progress
     text: pending pause task
@@ -227,8 +228,8 @@ Add:
 
 Update the note on the first step of the algorithm by adding the following bullet:
 
-> * Updating the [=animation trigger state|state=] of [=animation triggers=]
->   by running the [=updating animation trigger state=] procedure.
+> *   Updating the [=animation trigger state|state=] of [=animation triggers=]
+>     by running the [=updating animation trigger state=] procedure.
 
 <h3 id="animations">Animations</h3>
 
@@ -885,11 +886,11 @@ Replace:
 >
 >            : If |has finite timeline| is true,
 >            ::
->                Set |animation|'s [=start time=] to |seek time|.
+>                Set |animation|'s [=animation/start time=] to |seek time|.
 >
 >            : Otherwise,
 >            ::
->                Set |animation|'s [=hold time=] to |seek time|.
+>                Set |animation|'s [=animation/hold time=] to |seek time|.
 >
 >             </div>
 
@@ -1062,8 +1063,6 @@ Finally, the description of the [=end delay=] is updated to:
 >     An <a>end delay</a> may also be specified but is primarily only of
 >     use when sequencing animations such as by using a <a>sequence effect</a>.
 
-</div>
-
 The normative description is updated as follows:
 
 >     The lower bound of the <a>active interval</a> is defined by the
@@ -1508,7 +1507,7 @@ of an <a>animation effect</a> is as follows:
     This differs from the [=playback rate=] on an <a>animation</a>.
     Setting a [=playback rate=] on
     an <a>animation</a> to a negative value at a moment before the
-    <a>animation</a>'s </a>associated effect</a> has begun, will result in the
+    <a>animation</a>'s <a>associated effect</a> has begun, will result in the
     <a>associated effect</a> never playing.
 
 *   Setting the [=animation effect/playback rate=]
@@ -1688,13 +1687,13 @@ Update the definition to include the additional steps to for each phase:
 >                 and that parent group is in the <a>before
 >                 phase</a>, and the <a>fill mode</a> of this animation effect
 >                 is <span class="prop-value">none</span> or <span
->                 class="prop-value">forwards</a>,
+>                 class="prop-value">forwards</span>,
 >             ::  Return an <a>unresolved</a> <a>time value</a>.
 >             :   If the animation effect has a <a>parent group</a>
 >                 and that parent group is in the <a>after
 >                 phase</a>, and the <a>fill mode</a> of this animation effect
 >                 is <span class="prop-value">none</span> or <span
->                 class="prop-value">backwards</a>,
+>                 class="prop-value">backwards</span>,
 >             ::  Return an <a>unresolved</a> <a>time value</a>.
 >             :   Otherwise,
 >             ::  Return the result of evaluating
@@ -1821,7 +1820,6 @@ A simple example is illustrated below.
 When a <a>group effect</a> is <a>directly associated with
 an animation</a>, the <a>animation effect</a> associated with the
 <a>group effect</a> can be seeked, paused, and stopped as a unit.
-</p>
 
 </div>
 
@@ -2486,7 +2484,7 @@ Depending on its most recent [=animation trigger state|state=] |state|, the
 
     1. Let the [=animation trigger|trigger=] of |animation| be |new trigger|.
 
-    1. Run the procedure to [=updating animation trigger state=] for |trigger|.
+    1. Run the procedure to [=updating animation trigger state=] for |new trigger|.
   </div>
 
 ### Setting the timeline of an Animation Trigger ### {#setting-trigger-timeline}
@@ -2527,7 +2525,7 @@ Depending on its most recent [=animation trigger state|state=] |state|, the
       ::
 
         <dl class=switch>
-          : If |type| is 'animation-trigger-type/once'' and |did trigger| flag is true,
+          : If |type| is ''animation-trigger-type/once'' and |did trigger| flag is true,
           ::
             Then abort this procedure.
 
@@ -3157,7 +3155,7 @@ a {{EffectTiming}} object.</p>
 The <dfn lt="next sibling not included">next sibling of
 <var>effect</var> not included</dfn> in a set of <a>animation
 effects</a>, <var>effects</var> is determined using the following
-steps:</p>
+steps:
 
 1.  Let <var>context effect</var> be <var>effect</var>.
 2.  While the <a>next sibling</a> of <var>context effect</var> is not
@@ -3192,7 +3190,7 @@ To <dfn lt="insert children">insert</dfn> a series of zero or
 more <a>animation effects</a>, <var>effects</var>, to
 <var>parent</var>'s list of <a>child effects</a> before
 <var>reference child</var> perform the following steps for each
-<var>effect</var> in <var>effects</var>:</p>
+<var>effect</var> in <var>effects</var>:
 
 1.  <a lt="remove an animation effect">Remove</a> <var>effect</var>
     from its parent.
@@ -3464,10 +3462,10 @@ with which it is associated is updated.
 
 <h3 id="the-animatable-interface">The <code>Animatable</code> interface</h3>
 
-<div class="methods">
+<div dfn-type=method class=methods
+       dfn-for="Animatable">
 
-:   <dfn method for=Animatable lt="getAnimations()">
-    sequence&lt;Animation&gt; getAnimations()</dfn>
+:   <dfn lt="getAnimations(options)">sequence&lt;Animation&gt; getAnimations(|options|)</dfn>
 ::  Add:
 
     > If this object is the [=effect target/target element=] of two or more
@@ -3489,7 +3487,7 @@ is modified to add the following members:
   : <dfn lt="animate(keyframes, options)">Animation animate(|keyframes|, |options|)</dfn>
   ::  Amend step 6 of the procedure as follows:
 
-	6. If |options| is a {{KeyframeAnimationOptions}} object,
+    6. If |options| is a {{KeyframeAnimationOptions}} object,
         let |trigger| be the <code>trigger</code> member of |options|
         or, if <code>trigger</code> member of |options| is missing,
         a newly created {{AnimationTrigger}} with an empty
@@ -3498,6 +3496,8 @@ is modified to add the following members:
     7. Run the procedure to [=setting a trigger of an animation=] for |animation|
         with the |auto-rewind| flag set to true.
 
+</div>
+
 The {{KeyframeAnimationOptions}} dictionary interface
 is modified to add the following members:
 
@@ -3537,7 +3537,7 @@ partial dictionary KeyframeAnimationOptions {
 
 :   <dfn dict-member for=KeyframeAnimationOptions>trigger</dfn>
 ::  If present, specifies the <a>animation</a>’s associated [=animation trigger=]
-	by running the procedure of [=setting a trigger of an animation=].
+    by running the procedure of [=setting a trigger of an animation=].
 
 </div>