Skip to content

dom: Stub out the IntersectionObserver interface#33989

Merged
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:intersection-observer-stub
Oct 25, 2024
Merged

dom: Stub out the IntersectionObserver interface#33989
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:intersection-observer-stub

Conversation

@mrobinson
Copy link
Copy Markdown
Member

This is the first step toward implementing the IntersectionObserver
interface. It adds stubs which are exposed when a preference is turned
on. This is enough to get some sites with IntersectionObserver to
start working.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes are part of Implement IntersectionObserver #31021.
  • These changes do not require tests this change is still too early to test. Later when we start implementing IntersectionObsersver we can turn on tests.

@mrobinson mrobinson requested a review from gterzian as a code owner October 24, 2024 11:49
@mrobinson mrobinson force-pushed the intersection-observer-stub branch 2 times, most recently from 4c896cc to 56a35e3 Compare October 24, 2024 17:30
@mrobinson mrobinson enabled auto-merge October 24, 2024 17:31
@mrobinson mrobinson force-pushed the intersection-observer-stub branch from 56a35e3 to 4778326 Compare October 25, 2024 07:32
This is the first step toward implementing the IntersectionObserver
interface. It adds stubs which are exposed when a preference is turned
on. This is enough to get some sites with `IntersectionObserver` to
start working.

Signed-off-by: Martin Robinson <[email protected]>
@mrobinson mrobinson force-pushed the intersection-observer-stub branch from 4778326 to ad675ad Compare October 25, 2024 09:01
@mrobinson mrobinson added this pull request to the merge queue Oct 25, 2024
Merged via the queue into servo:main with commit 3b5dc06 Oct 25, 2024
@mrobinson mrobinson deleted the intersection-observer-stub branch October 25, 2024 10:27
@gterzian gterzian mentioned this pull request Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants