Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

What about supporting Intersection Observer? #64

Open
sergeymorkovkin opened this issue Apr 5, 2019 · 2 comments
Open

What about supporting Intersection Observer? #64

sergeymorkovkin opened this issue Apr 5, 2019 · 2 comments

Comments

@sergeymorkovkin
Copy link

It might be a good idea to rewrite plugin with support of Intersection Observer.

  1. It's got some support already - https://caniuse.com/#feat=intersectionobserver
  2. There is a polyfill - https://www.npmjs.com/package/intersection-observer

In short, it's more performant than current implementation.

@silvestreh
Copy link
Owner

silvestreh commented Apr 5, 2019 via email

@sergeymorkovkin
Copy link
Author

The thing is OnScreen has really great API. To me it makes perfect sense to keep up the pace. You may take a look at yall.js. They seem to have it done right.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants