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 emailsawogbem...@google.com
>>>
>>> Explainerhttps://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 componentBlink>Animation
>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EAnimation>
>>>
>>> TAG reviewhttps://github.com/w3ctag/design-reviews/issues/994
>>>
>>> TAG review statusIssues 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://flagsAnimationProgressAPI
>>>
>>> Finch feature nameNone
>>>
>>> Non-finch justificationNone
>>>
>>> Requires code in //chrome?False
>>>
>>> Tracking bughttps://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.

Reply via email to