Intersection Observer review

[LJWatson]

The WebPlat WG has just published the FPWD of Intersection Observer, triggering wide review. Thanks to its time in incubation, the spec already has good interop, so we're hopeful of moving to CR reasonably quickly.

We'd welcome a review by the TAG. Feedback no later than Friday 27th October would be appreciated, feedback earlier even more so.

Please file issues on the Intersection Observer repo. Thank you.

[dbaron]

So one thing I need to investigate further is that it feels to me that intersection observer running before the animation frame callbacks seems wrong, I think it should run substantially after them, with a bunch of other stuff in between, since rAF is primarily a writing callback (so should be early, before the natural style/layout), whereas IntersectionObserver is primarily a reading callback (so should be late, after the natural style/layout). But I suspect that some of my concern there may be that I think where HTML describes the running of the animation frame callbacks being wrong. I think this requires a good bit of digging into (to see what current specs say, and how interoperable existing implementations are); it's related to our #72.

[plinss]

I have concerns about the rootMargin being a string based API. This should be using CSS Typed OM if not now, at least some point if the future. The spec should at a minimum have a note about a typed OM version of the margin.

[LJWatson]

Gentle ping to remind you that we're looking for issues to be filed by the end of next week. Thanks.

[slightlyoff]

Discussed on today's call. Sounds like the delivering timing needs a look.

/cc @szager-chromium

[torgo]

Discussed on today's telecon.

[dbaron]

I'd like to leave this open to monitor the progress on w3c/IntersectionObserver#263 and whatwg/html#3246 etc.

[slightlyoff]

Per today's F2F, now for me to discuss open issues with @szager-chromium

[dbaron]

... but I think, as I mentioned in w3c/IntersectionObserver#263 (comment), that the steps should be still later. I think this requires writing a bunch of tests to check what happens today, though...

[LJWatson]

@szager @dbaron what is the state of this now?

[dbaron]

We separated out the more general bits into #489, but we're perhaps still leaving this open to wait on w3c/IntersectionObserver#263.

[dbaron]

@plinss and I are looking at this in a breakout in our virtual "Cork" face-to-face.

So we were leaving this open for w3c/IntersectionObserver#263, which is in turn waiting on whatwg/html#3246, which is in turn waiting on w3c/csswg-drafts#5115. While it would be nice to fix that sequence of things, we probably don't need to leave this review open for it.

These are also all referenced from #489.

[dbaron]

OK, we discussed in mini-plenary and we're happy closing this issue at this point. Thanks for requesting TAG review... a long time ago. Sorry this took so long.

[LJWatson]

Thank you @dbaron and @plinss.