Intent to Implement and Ship: AnimationEvent.pseudoElement

38 views
Skip to first unread message

Xida Chen

unread,
May 1, 2018, 12:07:32 PM5/1/18
to blink-dev

Contact emails

animati...@chromium.org


Spec

https://drafts.csswg.org/css-animations/#interface-animationevent


Summary

AnimationEvent.pseudoElement is a string that starts with "::", containing the name of the pseudo-element the animation runs on. If the animation is running on the element, then it is an empty string.


Is this feature supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes.


Risks

Interoperability and Compatibility

Low: This feature has a mature spec.


Edge: No signals

Firefox: Shipped in M23

Safari: In development

Source: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent/pseudoElement



Ergonomics

Are there any other platform APIs this feature will frequently be used in tandem with? No

Could the default usage of this API make it hard for Chrome to maintain good performance (i.e. synchronous return, must run on a certain thread, guaranteed return timing)? No


Activation

Will it be challenging for developers to take advantage of this feature immediately, as-is? No

Would this feature benefit from having polyfills, significant documentation and outreach, and/or libraries built on top of it to make it easier to use? No, this is a tiny API change, there is no need for polyfills.


Is this feature fully tested by web-platform-tests?

There is no test coverage on wpt/ at this moment. This chromium CL adds some test cases for this API.


Entry on feature dashboard

https://www.chromestatus.com/feature/5112855204462592

Yoav Weiss

unread,
May 3, 2018, 7:05:54 AM5/3/18
to Xida Chen, blink-dev
On Tue, May 1, 2018 at 6:07 PM Xida Chen <xida...@chromium.org> wrote:

Contact emails

animati...@chromium.org


Spec

https://drafts.csswg.org/css-animations/#interface-animationevent


Summary

AnimationEvent.pseudoElement is a string that starts with "::", containing the name of the pseudo-element the animation runs on. If the animation is running on the element, then it is an empty string.


Can you expand a bit on the use case for this attribute and how you'd expect it to be used by developers?
 

Is this feature supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes.


Risks

Interoperability and Compatibility

Low: This feature has a mature spec.


Edge: No signals


Have you reached out to the Edge folks?
 

Firefox: Shipped in M23

Safari: In development

 
Do you have a WebKit bug link?
 

Source: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent/pseudoElement



Ergonomics

Are there any other platform APIs this feature will frequently be used in tandem with? No

Could the default usage of this API make it hard for Chrome to maintain good performance (i.e. synchronous return, must run on a certain thread, guaranteed return timing)? No


Activation

Will it be challenging for developers to take advantage of this feature immediately, as-is? No

Would this feature benefit from having polyfills, significant documentation and outreach, and/or libraries built on top of it to make it easier to use? No, this is a tiny API change, there is no need for polyfills.


Is this feature fully tested by web-platform-tests?

There is no test coverage on wpt/ at this moment. This chromium CL adds some test cases for this API.


Would you consider the tests added in the CL to be sufficient for test coverage for the feature?
 
--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CALRxhAtGX0j6L6xi4kFvyobUdH7EdmMoVQCHv0JATxaJR25mHA%40mail.gmail.com.

Xida Chen

unread,
May 3, 2018, 10:03:38 AM5/3/18
to yo...@yoav.ws, blink-dev
On Thu, May 3, 2018 at 7:05 AM Yoav Weiss <yo...@yoav.ws> wrote:


On Tue, May 1, 2018 at 6:07 PM Xida Chen <xida...@chromium.org> wrote:

Contact emails

animati...@chromium.org


Spec

https://drafts.csswg.org/css-animations/#interface-animationevent


Summary

AnimationEvent.pseudoElement is a string that starts with "::", containing the name of the pseudo-element the animation runs on. If the animation is running on the element, then it is an empty string.


Can you expand a bit on the use case for this attribute and how you'd expect it to be used by developers?
Currently we cannot tell what is the target of an animation if it is running on a pseudo element, and this API solves that use case. 
 

Is this feature supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes.


Risks

Interoperability and Compatibility

Low: This feature has a mature spec.


Edge: No signals


Have you reached out to the Edge folks?
Not yet, but we do have an agenda to talk to them about web animations. 
 

Firefox: Shipped in M23

Safari: In development

 
Do you have a WebKit bug link?
No, this info is from our regular meeting with Safari engineers. 
 

Source: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent/pseudoElement



Ergonomics

Are there any other platform APIs this feature will frequently be used in tandem with? No

Could the default usage of this API make it hard for Chrome to maintain good performance (i.e. synchronous return, must run on a certain thread, guaranteed return timing)? No


Activation

Will it be challenging for developers to take advantage of this feature immediately, as-is? No

Would this feature benefit from having polyfills, significant documentation and outreach, and/or libraries built on top of it to make it easier to use? No, this is a tiny API change, there is no need for polyfills.


Is this feature fully tested by web-platform-tests?

There is no test coverage on wpt/ at this moment. This chromium CL adds some test cases for this API.


Would you consider the tests added in the CL to be sufficient for test coverage for the feature?
Yes, the tests in that CL should have sufficient coverage. 

Xida Chen

unread,
May 3, 2018, 10:05:02 AM5/3/18
to yo...@yoav.ws, blink-dev
Sending from the right email address.

PhistucK

unread,
May 3, 2018, 12:42:41 PM5/3/18
to Xida Chen, Yoav Weiss, blink-dev
> Currently we cannot tell what is the target of an animation if it is running on a pseudo element, and this API solves that use case. 

This is a requirement, not a use case... :(
Why do you need to know the pseudo element target of an animation? I assume you do know the element target, just not the pseudo element target, but how would that be helpful?

PhistucK


sligh...@chromium.org

unread,
May 3, 2018, 1:30:51 PM5/3/18
to blink-dev, xida...@chromium.org, yo...@yoav.ws
LGTM
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.

Chris Harrelson

unread,
May 3, 2018, 1:45:14 PM5/3/18
to Alex Russell, blink-dev, Xida Chen, Yoav Weiss
LGTM2

To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/126788d7-b5a2-4c9e-a195-459e93b493d6%40chromium.org.

Yoav Weiss

unread,
May 3, 2018, 2:01:25 PM5/3/18
to Chris Harrelson, Alex Russell, blink-dev, Xida Chen
LGTM3
Reply all
Reply to author
Forward
0 new messages