Intent to Ship: CSSOM View smooth scroll API

203 views
Skip to first unread message

Yunjia (Sandra) Sun

unread,
Apr 6, 2017, 4:33:32 PM4/6/17
to blin...@chromium.org, Ali Juma

Contact emails

aj...@chromium.org, yma...@chromium.org, suny...@chromium.org


Spec

http://dev.w3.org/csswg/cssom-view/ (See sections 4, 5, 7, 12, and 13.)


Summary

This adds an optional ScrollOptions and an optional ScrollIntoViewOptions arguments to CSSOM scrolling APIs; these additional arguments allows specifying that a particular scroll should be performed smoothly rather than instantly. This also adds a scroll-behavior CSS property, which specifies whether scrolls performed on a scrolling box (via CSSOM scrolling APIs or via navigation) should be smooth or instant.


Link to “Intent to Implement” blink-dev discussion

Intent to Implement: CSSOM View smooth scroll API



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

Yes.


Interoperability and Compatibility Risk

Edge: They have this as "Under consideration" at https://developer.microsoft.com/en-us/microsoft-edge/platform/status/cssomviewsmoothscrollapi/?q=cssom

Firefox: Shipped

Safari: No signals

Web developers: Positive


OWP launch tracking bug

crbug.com/243871


Entry on the feature dashboard

https://www.chromestatus.com/features/5812155903377408

Dimitri Glazkov

unread,
Apr 7, 2017, 10:42:29 AM4/7/17
to Yunjia (Sandra) Sun, blin...@chromium.org, Ali Juma
LGTM1

Philip Jägenstedt

unread,
Apr 7, 2017, 12:15:08 PM4/7/17
to Dimitri Glazkov, Yunjia (Sandra) Sun, blin...@chromium.org, Ali Juma
Hi Yunjia,

Recently we've begun asking about shared test suites in Intent to Ship, and today I tweaked our template to include this section:

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

Please link to the test suite. If any part of the feature is not tested by web-platform-tests, please include links to issues, e.g.:

  • A web-platform-tests issue with the "infra" label explaining why a certain thing cannot be tested. (example)

  • A spec issue for some change that would make it possible to test. (example)

  • A Chromium issue to upstream some existing tests. (example)

Such issues are not blocking in an Intent to Ship.


I search for some keywords in web-platform-tests and didn't find any tests. That's OK at this stage, but can you describe how the feature is currently tested and file bugs that track getting that test coverage into web-platform-tests?

P.S. The REF for this is CSSOMSmoothScroll, which helps to see exactly what the API surface is.

Yunjia (Sandra) Sun

unread,
Jun 13, 2017, 2:53:54 PM6/13/17
to blink-dev, dgla...@chromium.org, suny...@google.com, aj...@google.com
Hi Philip,

I've uploaded a patch that includes the web-platform-test. This patch modifies the idl file and has already landed, so we hope to get this Intent-to-Ship LGTMed soon :)

Thanks!

Hayato Ito

unread,
Jun 13, 2017, 10:23:49 PM6/13/17
to Yunjia (Sandra) Sun, blink-dev, dgla...@chromium.org, aj...@google.com
Could you make sure that wpt covers the case where Shadow DOM is used?

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/71e97106-2429-4142-9deb-ca2b2a888eb9%40chromium.org.

Philip Jägenstedt

unread,
Jun 14, 2017, 7:53:04 AM6/14/17
to Hayato Ito, Yunjia (Sandra) Sun, blink-dev, dgla...@chromium.org, aj...@google.com
I've left a few comments on the review. With that and sufficient Shadow DOM coverage, LGTM2.

Chris Harrelson

unread,
Jun 14, 2017, 11:07:29 AM6/14/17
to Philip Jägenstedt, Hayato Ito, Yunjia (Sandra) Sun, blink-dev, Dimitri Glazkov, Ali Juma
LGTM3

To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYez5Wx2yH-nc0KrX%2B-EFOGU1hiB09%2B4hbWS75%3DDaHDcGw%40mail.gmail.com.

Reply all
Reply to author
Forward
0 new messages