Threshold vs height about sal

[mciastek]

Sure, I'm closing this PR. Reopen when needed.

[mciastek]

According to IntersectionObserver docs, threshold can be only a number or an array of numbers. Currently, sal.js doesn't support array of numbers. If you could create a demo with this particular issue, maybe I could help.

[florianbouvot]

Thank you for your feedback, I try to create a demo of my issue.

[florianbouvot]

@mciastek I can't create a demo but this is a real project with the issue : https://www.caveduvaldor.com

Open this link on a small devices view and look at the third section of the page, user think there is no content.
If we can set something like 100px it will always works but percentage value is an issue if the animated area is very height.

[mciastek]

The threshold doesn't accept absolute values, only relative ones - it's mentioned in docs. It's not possible to pass 100px. You can use only floats or integers here. I can suggest to apply a custom animation to this tall section. The issue here is translate value, which by default is set to 20% of the height. You can add you custom animation, that will use translate with fixed value, like 100px. Then this section will be shifted down by 100px and user will need to scroll only 100px to see animation. Let me know if it helps you.

[florianbouvot]

@mciastek thank you, I'll try to use a fixed offset ;)