Skip to content

Commit 29d5056

Browse files
authored
Merge pull request w3c#7857 from w3c/active-animation
Clarify checking for active animations
2 parents 3d77ff7 + 5cb9f40 commit 29d5056

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

css-view-transitions-1/Overview.bs

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -893,15 +893,22 @@ The {{ViewTransition/domUpdated}} [=getter steps=] are to return [=this's=] [=Vi
893893
To <dfn>update transition DOM</dfn> given a {{ViewTransition}} |transition|:
894894

895895
1. Let |document| be |transition|'s [=relevant global object's=] [=associated document=].
896-
897-
1. For each [=page-transition pseudo-elements=] associated with |transition|,
898-
check whether there is an active animation associated with this pseudo-element.
899-
900-
Issue: Define what active animation means here.
901-
902-
1. If no [=page-transition pseudo-elements=] has an active animation:
903-
904-
Issue: There needs to be a definition/link for "active animation".
896+
897+
1. Let |hasActiveAnimations| be a boolean, initially false. For each [=page-transition pseudo-elements=]
898+
associated with |transition|:
899+
900+
1. Let |element| be the [=page-transition pseudo-element=].
901+
902+
1. For each |animation| that contains at least one [=animation effect=]
903+
whose [=effect target=] is |element| and whose associated [=animation timeline=]
904+
is of type [=document timeline=], set |hasActiveAnimations| to true if any of the
905+
following conditions is true:
906+
907+
1. |animation| is in <a lt="paused play state">paused</a> or <a lt="running play state">running</a> state.
908+
909+
1. [=pending-animation-event-queue=] has any events associated with |animation|.
910+
911+
1. If |hasActiveAnimations| is false:
905912

906913
1. Set |transition|'s [=ViewTransition/phase=] to "`done`".
907914

@@ -1022,13 +1029,6 @@ The {{ViewTransition/domUpdated}} [=getter steps=] are to return [=this's=] [=Vi
10221029

10231030

10241031
1. Set |transition|'s [=ViewTransition/phase=] to "`animating`".
1025-
1026-
Issue: How are keyframes scoped to user-agent origin? We could decide
1027-
scope based on whether `animation-name` in the computed style
1028-
came from a developer or UA stylesheet.
1029-
1030-
Issue: We should retarget the animation if computed properties for
1031-
incoming elements change.
10321032
</div>
10331033

10341034
## [=Create transition pseudo-elements=] ## {#create-transition-pseudo-elements-algorithm}

0 commit comments

Comments
 (0)