joshwnj / react-visibility-sensor

Sensor component for React that notifies you when it goes in or out of the window viewport.
MIT License
2.33k stars 195 forks source link

Cleanup and moving forward to 4.0 #120

Open joshwnj opened 6 years ago

joshwnj commented 6 years ago

Hi everyone, first up, please accept my apology for leaving this project in silence for a long time – sorry :) I know many of you are using this component and depending on it, so I don't take that lightly. Many have taken the time to report issues and create pull requests, and I appreciate your patience.

I'm going to put in some effort now to bring the project back to a healthy state, and also explore options of how to make that sustainable in the long term.

First up I'll be doing some triage on issues, to get a better understanding of where things are at. I'll also be merging some PRs that deal with updates to documentation and minor bugs.

Larger changes will be held just a little longer as more care needs to be taken towards backwards-compatibility.

And shortly I'll also be sharing my plans for a 4.0 release, and how you can help. Thanks!

joshwnj commented 6 years ago

Update on this: I am still preparing my plans for how to move things forward.

In the meantime if anyone wants to help, the thing we need the most is demos. Put something together on codepen / jsbin / codesandbox that demonstrates a certain use case of the component. Even better if this represents a use case in a real project you are responsible for.

If there is a certain feature or prop you depend on for your project, we want to see that covered by a demo, so we can prioritize features that are the most important to projects in the wild.

cjpete commented 5 years ago

Hi @joshwnj, not sure if you're still looking for examples or not. I've just found your library and started using it for an animate-on-visible style component.

I've added a crude version to Codesandbox here, and hope that it may be useful for you:

https://codesandbox.io/s/ppoo7yn5qj?fontsize=14

joshwnj commented 5 years ago

Hi @cjpete, that's a good example, thanks!