LGTM3

On 11/27/24 10:47 AM, Chris Harrelson wrote:
LGTM2

On Wed, Nov 27, 2024 at 7:42 AM Rick Byers <rby...@chromium.org> wrote:

    LGTM1 to ship in M133.

    It's nice to see the explainer and design improved based on TAG
    feedback, thank you for that!

    Rick

    On Tue, Nov 26, 2024 at 10:41 AM David Awogbemila
    <awogbem...@chromium.org> wrote:



        On Tue, Nov 26, 2024 at 10:35 AM David Awogbemila
        <awogbem...@chromium.org> wrote:


                    Contact emails

            awogbem...@google.com


                    Explainer

            https://github.com/DavMila/explainer-animation.progress


                    Specification

            
https://drafts.csswg.org/web-animations-2/#the-overall-progress-of-an-animation


                    Summary

            This feature adds an "overallProgress" property to the
            JavaScript class Animation[1]. The goal of this property
            is to provide authors a convenient and consistent
            representation of how far along an animation has advanced
            across its iterations and regardless of the nature of its
            timeline[2]. [1]
            https://developer.mozilla.org/en-US/docs/Web/API/Animation
            [2]
            https://developer.mozilla.org/en-US/docs/Web/API/AnimationTimeline



                    Blink component

            Blink>Animation
            
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EAnimation>


                    TAG review

            https://github.com/w3ctag/design-reviews/issues/994


                    TAG review status

            Issues addressed


                    Risks



                    Interoperability and Compatibility

            None



            /Gecko/: No signal
            (https://github.com/mozilla/standards-positions/issues/1073)

            /WebKit/: No signal
            (https://github.com/WebKit/standards-positions/issues/398)

            /Web developers/: No signals

            /Other signals/:


                    WebView application risks

            Does this intent deprecate or change behavior of existing
            APIs, such that it has potentially high risk for Android
            WebView-based applications?

            None



                    Debuggability

            Like other fields in the Animation class, overallProgress
            can be inspected with DevTools.



                    Will this feature be supported on all six Blink
                    platforms (Windows, Mac, Linux, ChromeOS, Android,
                    and Android WebView)?

            Yes


                    Is this feature fully tested by web-platform-tests
                    
<https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>?

            Yes

            
https://wpt.fyi/results/web-animations/interfaces/Animation/overallProgress.tentative.html
            
https://wpt.fyi/results/web-animations/interfaces/Animation/scroll-timeline-overallProgress.tentative.html



            Note that those test results might be showing as failing
            for a brief period because the feature was renamed from
            "progress" to "overallProgress" recently, and while the
            WPTs have immediately picked up this change, the version
            of Chrome against which they are run has yet to be
            similarly updated. However, you can see from the history
            
<https://wpt.fyi/results/web-animations/interfaces/Animation?run_id=5103325284663296>
            that past runs of the tests were passing:
            progress.tentative.html
            
<https://wpt.fyi/results/web-animations/interfaces/Animation/progress.tentative.html?run_id=5103325284663296>
 &
            scroll-timeline-progress.tentative.html
            
<https://wpt.fyi/results/web-animations/interfaces/Animation/scroll-timeline-progress.tentative.html?run_id=5103325284663296>.



                    Flag name on about://flags

            AnimationProgressAPI


                    Finch feature name

            None


                    Non-finch justification

            None


                    Requires code in //chrome?

            False


                    Tracking bug

            https://crbug.com/40914396


                    Sample links

            https://codepen.io/bramus/pen/BaXwmyZ
            https://codepen.io/awogbemila/pen/oNKpXWy
            https://davmila.github.io/demo-animation.progress/tda/index.html
            https://davmila.github.io/demo-animation.progress/sda/index.html


                    Estimated milestones

            No milestones specified

        We expect to ship this in M133. I've updated the Chromestatus
        entry.

-- 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+unsubscr...@chromium.org.
        To view this discussion visit
        
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF6P_GBZWKr_Ve2HqS%2BkXxfpEbHwY78o-s7ke9kSF3DbFw%40mail.gmail.com
        
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF6P_GBZWKr_Ve2HqS%2BkXxfpEbHwY78o-s7ke9kSF3DbFw%40mail.gmail.com?utm_medium=email&utm_source=footer>.

-- 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+unsubscr...@chromium.org.
    To view this discussion visit
    
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY-i_KDPix9JnPuZhFszZ1J-Mn8LD0z2Ae1TWWU696LPrg%40mail.gmail.com
    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY-i_KDPix9JnPuZhFszZ1J-Mn8LD0z2Ae1TWWU696LPrg%40mail.gmail.com?utm_medium=email&utm_source=footer>.

--
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+unsubscr...@chromium.org. To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw-f5S0gwh4ZgqD6G44bu31dX%3Dt7%3DXnRMq%2BM4FQCnYQ3_g%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw-f5S0gwh4ZgqD6G44bu31dX%3Dt7%3DXnRMq%2BM4FQCnYQ3_g%40mail.gmail.com?utm_medium=email&utm_source=footer>.

--
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+unsubscr...@chromium.org.
To view this discussion visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/bd823d60-ba2a-4198-b273-7fa5d42c200f%40chromium.org.

Reply via email to