Thanks for the WPT failure investigation and work to get them passing. LGTM1 once they pass. I'm excited to see this ship!
On Wed, Apr 5, 2023 at 8:40 PM David Awogbemila <awogbem...@chromium.org> wrote: > > > On Wed, Apr 5, 2023 at 11:24 AM Alex Russell <slightly...@chromium.org> > wrote: > >> Also, was this send to the TAG? >> > No this wasn't sent to the TAG since scrollend is already included in the > working group specification on scrolling ( > https://drafts.csswg.org/cssom-view/#scrolling-events) and it has already > shipped in Firefox like Yoav mentions. > > >> On Tuesday, April 4, 2023 at 9:11:26 PM UTC-7 Yoav Weiss wrote: >> >>> On Fri, Mar 31, 2023 at 3:50 PM David Awogbemila < >>> awogbem...@chromium.org> wrote: >>> >>>> Contact emails >>>> arg...@google.com, awogbem...@google.com >>>> >>>> Explainer >>>> https://github.com/argyleink/scrollend-explainer/blob/main/README.md >>>> >>> >>> The explainer says this shipped in 114. I guess it should say y'all are >>> expecting to ship at that point :) >>> >>> >>>> >>>> Specificationhttps://drafts.csswg.org/cssom-view/#scrolling-events >>>> >>>> Summary >>>> >>>> Scrollend events help developers reliably tell when a scroll has >>>> completed (including both the scroll itself and any updates to offsets from >>>> the scroll). Knowing when a scroll has completed is useful for various >>>> reasons, e.g. synchronizing some logic on the snapped section, fetching >>>> stuff in a list, triggering new animations, etc. This feature greatly >>>> simplifies the logic for handling end-of-scroll effects, ensuring that they >>>> are consistent across many different input modalities. Currently, >>>> developers address this need by observing scroll events and building ad-hoc >>>> timeout algorithms. >>>> >>>> >>>> Blink componentBlink>Scroll >>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EScroll> >>>> >>>> Search tagsscroll <https://chromestatus.com/features#tags:scroll> >>>> >>>> TAG review >>>> >>>> TAG review statusNot applicable >>>> >>> >>> Agree this is not needed, as this is following WG agreed-upon behavior, >>> that has already shipped in one implementation. >>> >>> >>>> >>>> >>>> Risks >>>> >>>> >>>> Interoperability and Compatibility >>>> >>>> >>>> >>>> *Gecko*: Shipped/Shipping ( >>>> https://bugzilla.mozilla.org/show_bug.cgi?id=1797013) >>>> >>>> *WebKit*: No signal ( >>>> https://github.com/WebKit/standards-positions/issues/150) There hasn't >>>> been an explicit position attached to the position request yet but there is >>>> a tracking WebKit issue: https://bugs.webkit.org/show_bug.cgi?id=201556 >>>> >>>> *Web developers*: Positive ( >>>> https://twitter.com/nghuuphuoc/status/1618806085158051846?s=20) Other >>>> examples: https://twitter.com/radogado/status/1621479592123826184?s=20 >>>> https://twitter.com/ebidel/status/1621037204297637891?lang=en >>>> >>>> *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? >>>> >>>> Not potentially high risk. >>>> >>>> >>>> Debuggability >>>> >>>> We verified via Protocol Monitor that DevTools supports breaking on >>>> scrollend listeners, similar to other events. DevTools UI change is needed >>>> to make this accessible which will be done via crrev.com/c/4376080. >>>> >>>> >>>> Will this feature be supported on all six Blink platforms (Windows, >>>> Mac, Linux, Chrome OS, 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/dom/events/scrolling?label=master&label=experimental&aligned >>> paints an odd picture, where our current experimental implementation passes >>> some of the tests, but not others, and Firefox seems to be failing some of >>> them. >>> Can you elaborate on the end state you're expecting once this ships? >>> >>> >>>> >>>> >>>> Flag nameN/A base::Feature is autogenerated from >>>> runtime_enabled_features.json5 >>>> <https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5;l=2920?q=%22name:%20%22ScrollEndEvents%22%22&sq=&ss=chromium%2Fchromium%2Fsrc> >>>> >>>> Requires code in //chrome?False >>>> >>>> Tracking bug >>>> https://bugs.chromium.org/p/chromium/issues/detail?id=907601 >>>> >>>> Estimated milestones >>>> M114 >>>> >>>> Anticipated spec changes >>>> >>>> Open questions about a feature may be a source of future web compat or >>>> interop issues. Please list open issues (e.g. links to known github issues >>>> in the project for the feature specification) whose resolution may >>>> introduce web compat/interop risk (e.g., changing to naming or structure of >>>> the API in a non-backward-compatible way). >>>> >>>> >>>> Link to entry on the Chrome Platform Status >>>> https://chromestatus.com/feature/5186382643855360 >>>> >>>> This intent message was generated by Chrome Platform Status >>>> <https://chromestatus.com/>. >>>> >>>> -- >>>> 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 on the web visit >>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF7nGNT0bwM8VY3Jj0TAEe9jNptKuwrMN1%3DO8tnqH2t8JQ%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF7nGNT0bwM8VY3Jj0TAEe9jNptKuwrMN1%3DO8tnqH2t8JQ%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 on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF5nehZf%3DZ3SzVtTSjgQpRENCh_G4aK1PhzO6ehtvQVTTQ%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAA6pwF5nehZf%3DZ3SzVtTSjgQpRENCh_G4aK1PhzO6ehtvQVTTQ%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 on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY8wEOs-gWwrL%3DpuXK%2BgOcA7hHpvG1X8ueXx%3Dh%3DzQ-QhFg%40mail.gmail.com.