1529389 - add a test for IntersectionObserverEntry.isIntersecting

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P3)

Description

[Cameron McCormack (:heycam)]

This is an interop issue between Chrome and Firefox. It looks like Firefox is following the spec and Chrome is not. It also looks like the MDN docs are out of date.

https://www.reddit.com/r/firefox/comments/asrsus/why_does_this_doesnt_work_on_firefox_correctly/

Comment 1

[Cameron McCormack (:heycam)]

Attachment: Bug 1529389 - Add a WPT for IntersectionObserverEntry.isIntersecting.

Comment 2

[Cameron McCormack (:heycam)]

(In reply to Cameron McCormack (:heycam) from comment #0)

It also looks like the MDN docs are out of date.

Specifically in the Thresholds section of https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API it talks about how isIntersecting takes the threshold into account, but actually it should be set to true or false just based on whether there is any intersection (or adjacency) at all, regardless of the threshold.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e291cb4e39c7
Add a WPT for IntersectionObserverEntry.isIntersecting. r=dholbert

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/e291cb4e39c7

Comment 5

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/15545 for changes under testing/web-platform/tests

Comment 6

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged