Not compatible with safari

These two lines not works in safari, because safari doesn’t respect document.documentElement.scrollTop/Left

https://github.com/terwanerik/ScrollTrigger/blob/56b4b04695ad3be2d69aa406cfd88746d194192f/ScrollTrigger.js#L548-L549

This is a complicated topic, so I just put a reference here, it is up to you to decide choose which solution: https://stackoverflow.com/questions/20514596/document-documentelement-scrolltop-return-value-differs-in-chrome/33462363#33462363

[terwanerik]

Ah alright i see, yeah the bindElement is used when you want to trigger classes inside a scrolling div.

For example (untested code);

<div class="wrap">
  <div data-scroll></div>
  <div data-scroll></div>
  <div data-scroll></div>
</div>

.wrap {
  height: 500px;
  overflow-y: scroll;
}

.wrap div[data-scroll] {
  height: 300px;
  width: 100%;
  background: red;
}

var wrapper = document.querySelector('.wrap');

/** 
  * First parameter = options
  * Second parameter = bindElement (where to fetch the [data-scroll] elements from)
  * Last parameter = the element to listen for `scroll` event, so the custom wrapper
  */
new ScrollTrigger({}, document.body, wrapper);

It then adds the scroll listener to the .wrap div, and checks the scrollTop from .wrap. In most cases you’d leave bindElement alone.

[nightire]

@terwanerik Ah, that makes sense now. Thank you!